Skip to content

ci: declare contents: read permissions on cifuzz workflow#1194

Open
arpitjain099 wants to merge 1 commit into
sigstore:mainfrom
arpitjain099:chore/cifuzz-perms
Open

ci: declare contents: read permissions on cifuzz workflow#1194
arpitjain099 wants to merge 1 commit into
sigstore:mainfrom
arpitjain099:chore/cifuzz-perms

Conversation

@arpitjain099
Copy link
Copy Markdown

Adds a top-level permissions: block declaring contents: read on .github/workflows/cifuzz.yaml.

The CIFuzz fuzz-testing workflow runs on pull requests and pushes to perform fuzz coverage; it does not push commits, create releases, comment on PRs, or call any GitHub write API. contents: read is therefore the minimum sufficient token scope.

I previously opened a related PR (#1188) for the gradle-wrapper-validation workflow in this repo. This is the same shape applied to the fuzz workflow, kept as a separate PR so each can be reviewed in isolation.

A short list of reasons this is cheap-but-worth-doing:

  • The OpenSSF Scorecard Token-Permissions check flags any workflow that does not declare explicit perms.
  • After the tj-actions/changed-files supply-chain incident (CVE-2025-30066, March 2025), explicit minimum permissions are one of the cheaper defenses against compromised third-party actions.
  • GitHub's own hardening guide recommends this pattern.

Validated locally with python3 -c "import yaml; yaml.safe_load(open('.github/workflows/cifuzz.yaml'))". No other edits.

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.

1 participant