Skip to content

Limit Bazel release workflow permissions#639

Open
Alb3e3 wants to merge 1 commit into
google:mainfrom
Alb3e3:limit-bcr-release-permissions
Open

Limit Bazel release workflow permissions#639
Alb3e3 wants to merge 1 commit into
google:mainfrom
Alb3e3:limit-bcr-release-permissions

Conversation

@Alb3e3

@Alb3e3 Alb3e3 commented Jun 10, 2026

Copy link
Copy Markdown

Summary

  • remove id-token: write and attestations: write from the Bazel Central Registry release path
  • keep contents: write, which is still needed for the BCR publish workflow to create release-side changes
  • align the caller and reusable release workflow permissions with the existing attest: false setting

Security rationale

The Bazel release path passes attest: false to bazel-contrib/publish-to-bcr, so this workflow does not need GitHub OIDC token minting or GitHub artifact attestation write access. Dropping those scopes reduces the privileges available to the release job while preserving the publish token flow used for BCR updates.

Validation

  • git diff --check
  • /tmp/actionlint-bin/actionlint .github/workflows/release.yml .github/workflows/release-bazel.yml
  • uvx zizmor --min-severity medium --min-confidence medium --format json .github/workflows/release.yml .github/workflows/release-bazel.yml

Note: zizmor still reports pre-existing unpinned action references in these release workflows. The excessive-permissions finding for this path is removed by this patch.

@Alb3e3 Alb3e3 marked this pull request as ready for review June 10, 2026 14:11
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