Skip to content

tighten GITHUB_TOKEN workflow permissions - #64

Merged
atkrad merged 1 commit into
mainfrom
fix-scorecard-token-permissions
Aug 3, 2026
Merged

tighten GITHUB_TOKEN workflow permissions#64
atkrad merged 1 commit into
mainfrom
fix-scorecard-token-permissions

Conversation

@atkrad

@atkrad atkrad commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Move OSV-Scanner security-events: write from workflow top-level to the calling job so Scorecard Token-Permissions stops flagging write scopes at the top level.
  • Drop unused packages: write from the release job (images go to Docker Hub, not GHCR).

Related

Fixes #65

Test plan

  • Confirm OSV-Scanner PR/scheduled workflows still upload SARIF
  • nix run .#lint / pre-commit clean
  • Confirm Scorecard re-run clears the Token-Permissions alerts on the OSV workflows and unused packages: write
  • Spot-check next tag/release job still creates the GitHub Release (needs contents: write)

Labels

  • One of: kind/feature, kind/bug, kind/docs, kind/chore
  • Add breaking-change if this breaks existing CLI or config behavior
  • Add skip-changelog for internal-only PRs that should not appear in notes

Checklist

  • Title is short and imperative (matches commit style)
  • Docs / CLI help updated when user-facing (README.md, docs/cli/) — N/A
  • No secrets or local-only paths in the diff

@github-actions github-actions Bot added the area/ci GitHub Actions and release packaging label Aug 3, 2026
@atkrad atkrad added kind/chore Cleanup, deps, CI, or refactor with no user-facing change skip-changelog Exclude this PR from release notes labels Aug 3, 2026
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@atkrad
atkrad merged commit 2398967 into main Aug 3, 2026
17 checks passed
@atkrad
atkrad deleted the fix-scorecard-token-permissions branch August 3, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci GitHub Actions and release packaging kind/chore Cleanup, deps, CI, or refactor with no user-facing change skip-changelog Exclude this PR from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: tighten GITHUB_TOKEN permissions for Scorecard

1 participant