Skip to content
This repository was archived by the owner on Jun 5, 2026. It is now read-only.

chore: pin GitHub Actions to commit SHAs - #282

Open
BGos87 wants to merge 1 commit into
mainfrom
chore/pin-github-actions-20260511
Open

chore: pin GitHub Actions to commit SHAs#282
BGos87 wants to merge 1 commit into
mainfrom
chore/pin-github-actions-20260511

Conversation

@BGos87

@BGos87 BGos87 commented May 11, 2026

Copy link
Copy Markdown

Summary

Pin every uses: ref in .github/workflows/ (and any composite action
files) to a full 40-character commit SHA, with the original tag
preserved as a # vX comment.

Why

Tags and branches are mutable, so a compromised action can replace what
runs in our pipelines without changing the tag we reference. Pinning to
a SHA closes that supply-chain vector. See GitHub's hardening guide:
https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#using-third-party-actions.

Deadline

TechOps is enforcing SHA-pinned GitHub Actions across the org by
June 8, 2026.
Merging this PR brings the repo into compliance ahead
of the cut-over; after that date workflows that still reference
mutable tags or branches will be blocked from running.

How

Generated mechanically with pinact run.
No version bumps were applied (strict pin); follow-up upgrades can come
from Renovate or a separate pinact run -u PR.

Test plan

  • CI green on this branch

Note

Low Risk
Low risk mechanical CI change that only pins actions/checkout to a specific commit for supply-chain hardening; workflow behavior should remain effectively the same aside from using an exact action revision.

Overview
Pins actions/checkout from the mutable @v3 tag to a specific commit SHA (# v3.6.0) across the build, lint, links, and case GitHub Actions workflows to improve supply-chain security and reproducibility.

Reviewed by Cursor Bugbot for commit c596e96. Bugbot is set up for automated code reviews on this repo. Configure here.

Pin every `uses:` ref in .github/workflows and composite actions to a
full 40-character commit SHA, with the original tag preserved as a
comment, e.g.

    uses: actions/checkout@11bd719 # v4

Tags and branches are mutable; commit SHAs are not. Pinning to a SHA
closes a supply-chain vector where a compromised action could replace
what runs in CI without changing the tag we reference.

Generated mechanically with `pinact run`
(https://github.com/suzuki-shunsuke/pinact). No version bumps were
applied (strict pin).
@vercel

vercel Bot commented May 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
doc-goquorum Ready Ready Preview, Comment May 11, 2026 2:54pm

Request Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c596e96. Configure here.

contents: read
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744 # v3.6.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mutable @main refs left unpinned across all workflows

High Severity

This PR claims to "pin every uses: ref" and bring the repo into TechOps compliance, but all four workflow files still reference ConsenSys/docs-gha/*@main — a mutable branch ref that is the same class of supply-chain risk this PR exists to close. Per the PR description, workflows referencing mutable tags or branches will be blocked from running after June 8, 2026, so these workflows will break at that deadline.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c596e96. Configure here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant