Skip to content

fix: add read permissions to GitHub actions workflow#1735

Merged
YamatoSecurity merged 1 commit intomainfrom
fix-actions-permissions
Jan 31, 2026
Merged

fix: add read permissions to GitHub actions workflow#1735
YamatoSecurity merged 1 commit intomainfrom
fix-actions-permissions

Conversation

@fukusuket
Copy link
Collaborator

What Changed

Added actions workflow permissions
I’d appreciate it if you could check it when you have time🙏

@fukusuket fukusuket requested a review from Copilot January 31, 2026 04:10
@fukusuket fukusuket self-assigned this Jan 31, 2026
@fukusuket fukusuket added the bug Something isn't working label Jan 31, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds explicit minimal GITHUB_TOKEN permissions to several GitHub Actions workflows to ensure they can read repository contents while following least-privilege guidance.

Changes:

  • Set workflow-level permissions: contents: read in multiple workflows.
  • Applies the same permission baseline across CI, integration, release automation, and timeline diff workflows.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/timeline-diff.yml Adds explicit contents: read permissions for the timeline diff workflow.
.github/workflows/rust.yml Adds explicit contents: read permissions for the Rust PR CI workflow.
.github/workflows/release.yml Adds explicit contents: read permissions for the release automation workflow.
.github/workflows/integration-test.yml Adds explicit contents: read permissions for integration test workflow runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@YamatoSecurity YamatoSecurity left a comment

Choose a reason for hiding this comment

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

@fukusuket LGTM! Thanks!

@YamatoSecurity YamatoSecurity merged commit 4ebff4a into main Jan 31, 2026
14 checks passed
@fukusuket fukusuket deleted the fix-actions-permissions branch January 31, 2026 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants