Skip to content

ci: declare contents:read on Continuity CI workflow#286

Open
arpitjain099 wants to merge 1 commit into
containerd:mainfrom
arpitjain099:chore/ci-permissions
Open

ci: declare contents:read on Continuity CI workflow#286
arpitjain099 wants to merge 1 commit into
containerd:mainfrom
arpitjain099:chore/ci-permissions

Conversation

@arpitjain099
Copy link
Copy Markdown

Pins ci.yml to contents: read at workflow scope. The three jobs (project, tests, cross) only check out the repo, set up Go, and run make test, make root-test, make build binaries across the matrix. No GitHub API write.

Defense-in-depth motivation is CVE-2025-30066 on tj-actions/changed-files: a compromised third-party action runs inside the existing job context and exfiltrates the workflow GITHUB_TOKEN via build logs. The blast radius equals the token's issued scope. Containerd's project-checks action and golangci/setup-go are third-party, so the cap is meaningful here.

Style matches the per-job permissions block in codeql.yml (security-events: write). YAML validated locally with yaml.safe_load.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants