Skip to content

Explicitly set permissions on all GitHub Actions workflows#1384

Merged
chris-rock merged 1 commit intomainfrom
fix/explicit-workflow-permissions
Feb 6, 2026
Merged

Explicitly set permissions on all GitHub Actions workflows#1384
chris-rock merged 1 commit intomainfrom
fix/explicit-workflow-permissions

Conversation

@chris-rock
Copy link
Copy Markdown
Member

Summary

  • Adds explicit permissions blocks to all workflows that were missing them, following the principle of least privilege
  • Removes the GitHubSecurityLab/actions-permissions/monitor step from lint.yaml since it has served its purpose
  • Workflows that already had job-level permissions (e.g. publish.yaml) now also have a restrictive top-level default

Workflows updated

Workflow Permissions
lint.yaml contents: read
link-check.yaml contents: read
leftover-spaces-cleaner.yaml contents: read
cla.yaml contents: read, pull-requests: write, issues: write, actions: read
publish-images.yaml contents: read
edge-integration-tests.yaml contents: read
cloud-tests.yaml contents: read, checks: write
release-manifests.yaml contents: write
publish.yaml contents: read (top-level default; jobs escalate individually)

Closes #843

Test plan

  • Verify lint workflow passes (only needs checkout + linter)
  • Verify CLA bot can still comment on PRs
  • Verify publish workflow can still push images and create releases
  • Verify cloud-tests can still publish test results

🤖 Generated with Claude Code

Adds top-level permissions blocks to every workflow that was missing one,
following the principle of least privilege. Also removes the now-unnecessary
permissions monitor action from the lint workflow.

Closes #843

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 6, 2026

Test Results

  5 files  ±0   41 suites  ±0   34m 26s ⏱️ + 1m 19s
301 tests ±0  301 ✅ ±0  0 💤 ±0  0 ❌ ±0 
320 runs  ±0  318 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 1735c77. ± Comparison against base commit feff81f.

@chris-rock chris-rock merged commit 637c4fa into main Feb 6, 2026
23 checks passed
@chris-rock chris-rock deleted the fix/explicit-workflow-permissions branch February 6, 2026 16:21
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Explicitly set linting permissions

1 participant