Skip to content

Commit 8c6f1fb

Browse files
committed
ci: declare read-all permissions on build-test workflow
The build-test workflow has two jobs: gen-matrix -- actions/checkout + tj-actions/changed-files + actions/github-script to compute the matrix from the PR diff. No GitHub API write. build -- actions/checkout + docker/build-push-action with `push: false, load: true`, then docker run/test. No GitHub API write, no registry push. `permissions: read-all` at workflow scope matches the shorthand scorecard.yml already uses, and brings build-test in line with the per-job blocks declared by official-pr.yml, missing-checksum.yml, shfmt.yml, doctoc.yml, eclint.yml, automatic-updates.yml, and markdown-link-check.yml. Note: this workflow uses tj-actions/changed-files, the action behind CVE-2025-30066 (the March-2025 supply-chain attack). The compromised release exfiltrated the runner's GITHUB_TOKEN through workflow logs; the downstream blast radius scaled with whatever scope the token had been issued at. Pinning the workflow to read-only bounds that radius. Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
1 parent f2e1347 commit 8c6f1fb

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

.github/workflows/build-test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ on:
1515
- genMatrix.js
1616
- ".github/workflows/build-test.yml"
1717

18+
permissions: read-all
19+
1820
jobs:
1921
gen-matrix:
2022
name: generate-matrix

0 commit comments

Comments
 (0)