Skip to content

ci: declare least-privilege workflow-level contents: read#2855

Merged
oscerd merged 1 commit into
apache:mainfrom
arpitjain099:chore/declare-workflow-perms
Jun 11, 2026
Merged

ci: declare least-privilege workflow-level contents: read#2855
oscerd merged 1 commit into
apache:mainfrom
arpitjain099:chore/declare-workflow-perms

Conversation

@arpitjain099

Copy link
Copy Markdown
Contributor

Hardens 1 workflow(s) in this repo by declaring a workflow-level permissions: contents: read. Today those workflows inherit the legacy broad read-write GITHUB_TOKEN; the read-only default reduces blast radius if any step is compromised.

I checked each file - they read the checkout and run tests/lints; no GitHub API writes (no gh pr/issue, no git push, no release/publish/comment actions). So behavior is unchanged.

Reference: the tj-actions/changed-files compromise (CVE-2025-30066) is the canonical reason to apply least-privilege defaults.

Declares an explicit workflow-level permissions: contents: read on 1 workflow that currently inherit the default broad read-write GITHUB_TOKEN. Each file was inspected and only reads the checkout; none publish, push, or write via the GitHub API. Post-CVE-2025-30066 hardening default.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@oscerd oscerd merged commit 838ae31 into apache:main Jun 11, 2026
2 of 5 checks passed
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