Skip to content

SEC-178: pin GitHub Action refs to full SHAs#35

Merged
kwent merged 1 commit intomasterfrom
security/sec-178-pin-actions
Apr 21, 2026
Merged

SEC-178: pin GitHub Action refs to full SHAs#35
kwent merged 1 commit intomasterfrom
security/sec-178-pin-actions

Conversation

@youngjk
Copy link
Copy Markdown
Contributor

@youngjk youngjk commented Apr 21, 2026

Summary

Pins all external GitHub Actions in this repo to full 40-char commit SHAs. Required before the org-wide sha_pinning_required = true policy (rootlyhq/terraform-rootly#891) lands — otherwise CI here fails validation on the next run after apply.

Linear: SEC-178 (follow-up to SEC-89).

Changes

File Actions pinned
.github/workflows/maven.yml actions/checkout@v6, actions/setup-java@v5
.github/workflows/publish.yml actions/checkout@v6, actions/setup-java@v5 (×2), actions/github-script@v9
.github/workflows/test.yml actions/checkout@v6, actions/setup-java@v5

All SHAs preserve the original tag as a trailing comment (Dependabot convention).

SHAs

  • actions/checkout@v6de0fac2e4500dabe0009e67214ff5f5447ce83dd
  • actions/setup-java@v5be666c2fcd27ec809703dec50e508c2fdc7f6654
  • actions/github-script@v93a2844b7e9c422d3c10d287c895573f7108da1b3

Resolved via gh api /repos/OWNER/REPO/commits/TAG on 2026-04-21.

Test plan

  • test.yml passes on the Java 17 matrix
  • maven.yml passes on Java 17 and 21 matrix
  • Next publish to Maven Central + GitHub Packages works end-to-end

🤖 Generated with Claude Code

Pins all external actions in maven.yml, publish.yml, and test.yml to
full commit SHAs of their v5/v6/v9 tags as of 2026-04-21.

Required before the org-wide sha_pinning_required policy
(rootlyhq/terraform-rootly#891) lands.

SHAs:
  actions/checkout@v6      -> de0fac2e4500dabe0009e67214ff5f5447ce83dd
  actions/setup-java@v5    -> be666c2fcd27ec809703dec50e508c2fdc7f6654
  actions/github-script@v9 -> 3a2844b7e9c422d3c10d287c895573f7108da1b3

Linear: SEC-178 (follow-up to SEC-89).
@kwent kwent merged commit 714893f into master Apr 21, 2026
3 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.

2 participants