Skip to content

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

Merged
ajanon merged 1 commit into
canonical:mainfrom
arpitjain099:chore/declare-workflow-perms
Jun 2, 2026
Merged

ajanon merged 1 commit into
canonical:mainfrom
arpitjain099:chore/declare-workflow-perms

Conversation

@arpitjain099

Copy link
Copy Markdown
Contributor

Small CI hardening: declares an explicit workflow-level permissions: contents: read on 2 workflow(s) that currently inherit the default broad read-write GITHUB_TOKEN.

I inspected each file before including it; none publish, push, comment on issues/PRs, or otherwise write via the GitHub API, so the read-only default does not change behavior. Workflows that need to write (stale, release, gh-pages-deploy, publish actions, etc.) are intentionally left out of this PR.

This is the post-CVE-2025-30066 hardening pattern for default token scope.

Declares an explicit workflow-level permissions: contents: read on 2 workflows 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>
@arpitjain099
arpitjain099 requested a review from a team as a code owner May 31, 2026 01:13

@ajanon ajanon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi!

This looks good to me, thank you for your contribution!

@ajanon
ajanon merged commit 7e3064d into canonical:main Jun 2, 2026
1 check 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