Skip to content

ci: pin workflow actions to commit SHAs (PROT-39)#3

Merged
BlakeHastings merged 1 commit into
mainfrom
ci/PROT-39-pin-workflow-actions
Jun 1, 2026
Merged

ci: pin workflow actions to commit SHAs (PROT-39)#3
BlakeHastings merged 1 commit into
mainfrom
ci/PROT-39-pin-workflow-actions

Conversation

@BlakeHastings
Copy link
Copy Markdown
Member

Summary

Supply-chain hardening for the public repo: pin every action in .github/workflows/release-please.yml to a full commit SHA (with a version comment) instead of a moving tag. A tag like @v4 can be repointed at any time; a SHA is immutable, so the workflow always runs the exact code we reviewed.

Jira: PROT-39

Changes

Action Before After
googleapis/release-please-action @v4 @5c625bf… # v4
actions/checkout @v4 @34e1148… # v4
actions/setup-dotnet @v5 @9a946fd… # v5

Majors are intentionally unchanged. Newer majors exist (actions/checkout v6, release-please-action v5); upgrading those is deferred to a separate, deliberate change so an untested major bump isn't bundled into a security pin.

Why this is the right hardening here

The repo is public and the release-please workflow runs with contents / issues / pull-requests write. Pinning to SHAs is the standard mitigation and pairs with the default_workflow_permissions: read baseline already in place.

Side effect (intended)

Merging this to main re-triggers the release-please workflow. Now that "Allow GitHub Actions to create and approve pull requests" is enabled, that run should finally open the 0.1.0 Release PR that the previous run couldn't create.

🤖 Generated with Claude Code

Pin actions/checkout (v4), actions/setup-dotnet (v5), and googleapis/release-please-action (v4) to immutable commit SHAs with version comments, closing the supply-chain risk of a moving tag being repointed on this public repo. Majors are unchanged; newer majors (checkout v6, release-please v5) are deferred as a separate evaluation.

Refs: PROT-39

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@BlakeHastings BlakeHastings merged commit 036befa into main Jun 1, 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.

2 participants