Skip to content

Conversation

@mortenson
Copy link
Contributor

This PR pins action references to commit hashes to mitigate supply chain attacks where a bad actor will push a new tag or override an existing tag, leading to us running malicious code immediately without explicitly updating.

We already use pinning in the backend, frontend, and infra repos.

Documentation on how to use pinact can be found here: https://developers.dmz.internal.mercury.com/docs/backend/pinning-github-action-hashes

9999years
9999years previously approved these changes Aug 22, 2025
Copy link
Member

@9999years 9999years left a comment

Choose a reason for hiding this comment

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

LGTM, is the CI failure significant? (The Garnix failures look spurious, we're out of quota.)

@mortenson
Copy link
Contributor Author

@9999years oops, since MercuryTechnologies/mercury-github-actions/pinact-check isn't public I shouldn't have included it in this PR, my bad. We should maybe move that to a public repo eventually.

@9999years 9999years merged commit 57f6aa0 into main Aug 22, 2025
0 of 2 checks passed
@9999years 9999years deleted the workflow-enforcer-1755725650057 branch August 22, 2025 21:49
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