Skip to content

Harden GitHub Actions token permissions#122

Merged
atimin merged 1 commit intoreductstore:mainfrom
mother-6000:121-least-privilege-github-token
Apr 30, 2026
Merged

Harden GitHub Actions token permissions#122
atimin merged 1 commit intoreductstore:mainfrom
mother-6000:121-least-privilege-github-token

Conversation

@mother-6000
Copy link
Copy Markdown
Contributor

Closes #121

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • CHANGELOG.md has been updated (for bug fixes / features / docs)

What kind of change does this PR introduce?

Security hardening / CI configuration update.

What was changed?

  • Set workflow-level default GITHUB_TOKEN permissions to read-all in .github/workflows/ci.yml.
  • Scoped elevated permissions only to the build job that uploads artifacts:
    • contents: read
    • actions: write
  • Kept all other jobs on read-only token permissions by default.

This implements the approved plan on issue #121 (plan comment: #121 (comment)).

Related issues

Does this PR introduce a breaking change?

No.

Other information:

Validation performed:

  • python3 YAML parse check for .github/workflows/ci.yml (syntax OK).

@atimin atimin merged commit 9e4e077 into reductstore:main Apr 30, 2026
14 checks 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.

Least-privilege GITHUB_TOKEN permissions

2 participants