Skip to content

fix: pin 1 unpinned action#3547

Open
dagecko wants to merge 1 commit intoventoy:masterfrom
dagecko:runner-guard/fix-ci-security
Open

fix: pin 1 unpinned action#3547
dagecko wants to merge 1 commit intoventoy:masterfrom
dagecko:runner-guard/fix-ci-security

Conversation

@dagecko
Copy link
Copy Markdown

@dagecko dagecko commented Mar 30, 2026

Re-submission of #3544. Had a problem with my fork and had to delete it, which closed the original PR. Apologies for the noise.

Summary

This PR pins a GitHub Action to an immutable commit SHA instead of a mutable branch reference.

  • Pin Yikun/hub-mirror-action@master to full 40-character SHA

How to verify

Review the diff, the change is mechanical and preserves workflow behavior:

  • SHA pinning: action@master becomes action@abc123 # master, original ref preserved as comment
  • No workflow logic, triggers, or permissions are modified

I've been researching CI/CD supply chain attack vectors and submitting fixes to affected repos. Based on that research I built a scanner called Runner Guard and open sourced it here so you can scan yourself if you want to. I'll be posting more advisories over the next few weeks on Twitter if you want to stay in the loop.

If you have any questions, reach out. I'll be monitoring comms.

- Chris (dagecko)

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.

1 participant