Skip to content

Commit de74b2f

Browse files
committed
security(ci): fix SARIF upload permissions and deprecation
- Add permissions: actions: read for workflow run metadata - Upgrade to github/codeql-action/upload-sarif@v4 - Guard uploads to skip forked PRs (avoid Resource not accessible by integration) - Applies to Trivy and Dependency-Check SARIF uploads
1 parent 72ccaad commit de74b2f

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

.github/workflows/security.yml

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ on:
1010
permissions:
1111
contents: read
1212
security-events: write
13+
actions: read
1314
pull-requests: write
1415

1516
jobs:
@@ -78,8 +79,12 @@ jobs:
7879
output: 'trivy-results.sarif'
7980

8081
- name: Upload Trivy results to GitHub Security
81-
uses: github/codeql-action/upload-sarif@v3
82-
if: always()
82+
uses: github/codeql-action/upload-sarif@v4
83+
if: >
84+
always() && (
85+
github.event_name != 'pull_request' ||
86+
github.event.pull_request.head.repo.full_name == github.repository
87+
)
8388
with:
8489
sarif_file: 'trivy-results.sarif'
8590
category: 'trivy'
@@ -178,8 +183,12 @@ jobs:
178183
--enable-retired
179184
180185
- name: Upload Dependency-Check results
181-
uses: github/codeql-action/upload-sarif@v3
182-
if: always()
186+
uses: github/codeql-action/upload-sarif@v4
187+
if: >
188+
always() && (
189+
github.event_name != 'pull_request' ||
190+
github.event.pull_request.head.repo.full_name == github.repository
191+
)
183192
with:
184193
sarif_file: 'dependency-check-report.sarif'
185194
category: 'dependency-check'

0 commit comments

Comments
 (0)