Skip to content

Conversation

@oreparaz
Copy link
Contributor

@oreparaz oreparaz commented Mar 20, 2025

We do not want a compromised github action (or maintainer thereof) to take over the repo's secrets or integrity. This PR pins every action to a full length commit SHA.

This is the way recommended by github [0]. The following tool was used in this PR: https://github.com/mheap/pin-github-action

Example vulnerability that exploits this vector: GHSA-mrrh-fwg8-r2c3 (CVE-2025-30066).

[0] https://docs.github.com/en/actions/security-for-github-actions/security-guides/security-hardening-for-github-actions#using-third-party-actions

@zakiali
Copy link
Collaborator

zakiali commented Mar 25, 2025

@oreparaz Do you mind resolving conflicts and then I can do a quick review after that

Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

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

great idea and long overdue

@oreparaz oreparaz force-pushed the oscar/2025-03-20/pin-github/action branch from 875c8ac to 1032876 Compare March 26, 2025 08:39
We do not want a compromised github action (or maintainer thereof) to take over
the repo's secrets or integrity. This PR pins every action to a full length
commit SHA.

This is the way recommended by github [0]. The following tool was used in this
PR: https://github.com/mheap/pin-github-action

[0] https://docs.github.com/en/actions/security-for-github-actions/security-guides/security-hardening-for-github-actions#using-third-party-actions
@oreparaz oreparaz force-pushed the oscar/2025-03-20/pin-github/action branch from 1032876 to b3d0eee Compare March 26, 2025 08:45
@oreparaz
Copy link
Contributor Author

👌 resolved conflicts and rebased @zakiali

@zakiali zakiali merged commit 49c5ec9 into block:main Mar 26, 2025
6 checks passed
ahau-square pushed a commit that referenced this pull request May 2, 2025
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
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.

3 participants