Skip to content

chore: pin GitHub Actions to commit SHAs#360

Merged
fspreiss merged 16 commits intomainfrom
chore/pin-actions-to-sha
Apr 10, 2026
Merged

chore: pin GitHub Actions to commit SHAs#360
fspreiss merged 16 commits intomainfrom
chore/pin-actions-to-sha

Conversation

@slawomirbabicz
Copy link
Copy Markdown
Contributor

Pin GitHub Actions to commit SHAs

GitHub Actions referenced by tag (e.g. actions/checkout@v4) use a mutable pointer — the tag owner can move it to a different commit at any time, including a malicious one. This is the attack vector used in the tj-actions/changed-files incident (CVE-2025-30066).

Pinning to a full 40-character commit SHA makes the reference immutable. The # tag comment preserves human readability so reviewers can tell which version is pinned.

Important: a SHA can also originate from a forked repository. A malicious actor can fork an action, push a compromised commit to the fork, and the SHA will resolve — but it won't exist in the upstream canonical repo. Each SHA in this PR was verified against the action's canonical repository (not a fork).

Changes

  • actions/checkout@v4 -> actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1

  • ZenVoich/setup-mops@v1 -> ZenVoich/setup-mops@3e94e453352269b34137b5ce49f09a8df81bed7d # v1.4.1

  • dfinity/conventional-pr-title-action@v4.0.0 -> dfinity/conventional-pr-title-action@41fbb167fd524835beda2b81882b7c75b1061dbc # v4.0.0

  • actions/setup-node@v4 -> actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0

  • actions/upload-pages-artifact@v3 -> actions/upload-pages-artifact@56afc609e74202658d3ffba0e8f6dda462b719fa # v3.0.1

  • actions/deploy-pages@v4 -> actions/deploy-pages@d6db90164ac5ed86f2b6aed7e0febac5b3c0c03e # v4.0.5

Files modified

  • .github/workflows/backend-motoko-format-check.yml
  • .github/workflows/backend-motoko.yml
  • .github/workflows/backend-rust-canbench.yml
  • .github/workflows/backend-rust.yml
  • .github/workflows/clippy.yml
  • .github/workflows/conventional-commits.yml
  • .github/workflows/examples-basic-bls-signing.yml
  • .github/workflows/examples-basic-ibe.yml
  • .github/workflows/examples-basic-timelock-ibe.yml
  • .github/workflows/examples-encrypted-chat.yml
  • .github/workflows/examples-encrypted-notes-dapp.yml
  • .github/workflows/examples-password-manager-with-metadata.yml
  • .github/workflows/examples-password-manager.yml
  • .github/workflows/frontend.yml
  • .github/workflows/publish-frontend-docs.yml
  • .github/workflows/publish-frontend.yml

Security warnings

  • FORK of aslafy-z/conventional-pr-title-action

@slawomirbabicz slawomirbabicz requested a review from a team as a code owner April 9, 2026 06:57
@fspreiss
Copy link
Copy Markdown
Contributor

fspreiss commented Apr 9, 2026

Thanks, @slawomirbabicz!

@fspreiss fspreiss merged commit d5b39d4 into main Apr 10, 2026
38 checks passed
@fspreiss fspreiss deleted the chore/pin-actions-to-sha branch April 10, 2026 09:06
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