Skip to content

ci: lock down token permissions, attest release binaries, add timeouts#73

Open
mr-raj12 wants to merge 2 commits into
zerodha:masterfrom
mr-raj12:mr-raj12-patch-1
Open

ci: lock down token permissions, attest release binaries, add timeouts#73
mr-raj12 wants to merge 2 commits into
zerodha:masterfrom
mr-raj12:mr-raj12-patch-1

Conversation

@mr-raj12
Copy link
Copy Markdown

@mr-raj12 mr-raj12 commented May 12, 2026

What changed

ci.yml:

  • permissions: contents: read at workflow level. Nothing writes to the repo by default.
  • concurrency group: new push to the same branch cancels the old run
  • paths filter: README/LICENSE changes no longer trigger CI
  • timeout-minutes: 20 so a stuck Nix install doesn't eat runner time for hours
  • All actions pinned to commit SHA instead of mutable tags

release.yml:

  • actions/attest-build-provenance after each of the 4 platform builds. Links each binary to this exact commit so users can verify what they downloaded came from this workflow.
  • id-token: write and attestations: write added to permissions (needed for OIDC signing)
  • timeout-minutes: 45
  • All actions pinned to commit SHA

Why

Anyone downloading a release binary today has no way to check it wasn't swapped after the build. The attestation steps fix that. SHA pinning and a read-only default token were both missing, and OSSF Scorecard flags both.

mr-raj12 added 2 commits May 13, 2026 04:20
Updated the release workflow to include binary attestations and specific version tags for actions.
@mr-raj12 mr-raj12 changed the title ci: add permissions hardening, SLSA attestations, and timeouts ci: lock down token permissions, attest release binaries, add timeouts May 12, 2026
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.

1 participant