Skip to content

Conversation

@picatz
Copy link
Contributor

@picatz picatz commented Oct 29, 2025

Summary

  • .github/workflows/nightly-throughput-stress.yml: Added a workflow-level permissions block restricting the token to contents: read, matching the read-only actions in the nightly throughput stress run.
  • .github/workflows/build-binaries.yml: Declared workflow-wide contents: read to cover checkout and artifact upload while removing implicit write access.
  • .github/workflows/ci.yml: Introduced top-level contents: read so CI, dependency checks, and reusable feature tests run with read-only repository access.
  • .github/workflows/run-bench.yml: Set workflow permissions to contents: read for local and reusable bench runs that only fetch repository code.
  • .github/workflows/nightly.yml: Added contents: read so the scheduled wrapper workflow passes least-privilege access to the reusable bench workflow.
  • .github/workflows/omes.yml: Specified contents: read for repository checkout and packages: write to allow the downstream reusable workflow to push container images.

This change was made by an automated process to ensure all GitHub Actions workflows have explicitly defined permissions as per best practices.
@picatz picatz requested a review from a team as a code owner October 29, 2025 16:01
@CLAassistant
Copy link

CLAassistant commented Oct 29, 2025

CLA assistant check
All committers have signed the CLA.

@picatz
Copy link
Contributor Author

picatz commented Oct 30, 2025

Small conflict was introduced after #1200 merged (which added a permissions block).

@tconley1428 tconley1428 merged commit 660ac2b into main Oct 30, 2025
24 of 27 checks passed
@tconley1428 tconley1428 deleted the security-campaign/set-explicit-github-actions-permissions branch October 30, 2025 18:29
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.

4 participants