Skip to content

chore(ci): apply least-privilege GITHUB_TOKEN permissions#216

Merged
atimin merged 1 commit into
mainfrom
209-least-privilege-github-token
Apr 30, 2026
Merged

chore(ci): apply least-privilege GITHUB_TOKEN permissions#216
atimin merged 1 commit into
mainfrom
209-least-privilege-github-token

Conversation

@mother-6000
Copy link
Copy Markdown
Contributor

Closes #209

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?

Chore / CI hardening

What was changed?

  • Set workflow-level default token permission to read-all in .github/workflows/ci.yml
  • Added job-level overrides only where elevated access is required:
    • build: actions: write + contents: read (artifact upload)
    • make_release: contents: write (create GitHub release)
    • upload_release: contents: write (upload release assets)
  • Kept all other jobs on read-only token scope.

Implementation follows the approved plan on issue #209:
#209 (comment)

Related issues

Does this PR introduce a breaking change?

No. This only tightens GitHub Actions token permissions.

Other information:

Before: no explicit permissions in workflow (relied on repository defaults).
After: explicit least-privilege defaults with narrowly scoped write permissions for release/artifact jobs only.

@atimin atimin merged commit 381d54a into main Apr 30, 2026
14 checks passed
@atimin atimin deleted the 209-least-privilege-github-token branch April 30, 2026 13:34
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