Skip to content

[CI][Security] Declare least-privilege GITHUB_TOKEN permissions - #359

Merged
rsamoilov merged 1 commit into
rage-rb:mainfrom
RedZapdos123:ci/least-privilege-permissions
Jul 27, 2026
Merged

[CI][Security] Declare least-privilege GITHUB_TOKEN permissions#359
rsamoilov merged 1 commit into
rage-rb:mainfrom
RedZapdos123:ci/least-privilege-permissions

Conversation

@RedZapdos123

@RedZapdos123 RedZapdos123 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Description:

Issue #358 reported that the test, extension, documentation, and changelog workflows inherited their GITHUB_TOKEN permissions from repository or organization defaults.

This PR gives the test, extension, and documentation workflows read only contents access. It gives the changelog workflow read only pull request access.

This keeps all other token permissions disabled and leaves the existing release workflow permissions unchanged.

Closes #358.

Checklist:

  • I have validated all workflow files using go run github.com/rhysd/actionlint/cmd/actionlint@latest.
  • I have validated the workflow YAML files using Ruby's YAML parser.

Declare the minimum GITHUB_TOKEN access required by each non-release workflow instead of relying on repository or organization defaults.

Signed-off-by: Mridankan Mandal <xerontitan90@gmail.com>
@RedZapdos123 RedZapdos123 changed the title ci: restrict workflow token permissions [CI][Security] Declare least-privilege GITHUB_TOKEN permissions Jul 24, 2026
@RedZapdos123

Copy link
Copy Markdown
Contributor Author

Since this is a CI only security hardening change. It does not affect Rage runtime behavior, public API, or any user facing functionality, so no changelog entry added.

@rsamoilov rsamoilov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thank you! And sorry for the delays with reviews.

@rsamoilov
rsamoilov merged commit 7ff9c4c into rage-rb:main Jul 27, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI][Security] Declare least-privilege GITHUB_TOKEN permissions

2 participants