Skip to content

Upgrade all third-party GitHub Actions and pin them to commit hashes#99

Merged
jeancochrane merged 4 commits into
masterfrom
copilot/upgrade-ccao-data-ptaxsim
Jun 1, 2026
Merged

Upgrade all third-party GitHub Actions and pin them to commit hashes#99
jeancochrane merged 4 commits into
masterfrom
copilot/upgrade-ccao-data-ptaxsim

Conversation

Copilot AI commented May 22, 2026

Copy link
Copy Markdown
Contributor

This PR upgrades all of our third-party GitHub Actions to ensure they are compatible with the upcoming Node 20 deprecation.

While we're at it, we also switch all of our references to third-party actions to point to commit hashes, so as to protect ourselves from the the ongoing scourge of supply chain attacks against third-party actions. These supply chain attacks work by pushing malicious code to running tags like v2, which will then cause downstream workflows like ours to pull in the malicious code the next time they run.

For all of our third-party actions, I sourced the commit hashes by checking the Releases page for the most recent release and copying the commit hash of that release. I also added inline comments indicating the version number that corresponds to the commit hash.

Test workflows to confirm these upgrades don't break anything:

Connects https://github.com/ccao-data/aws-infrastructure/issues/59.

@jeancochrane jeancochrane changed the title Upgrade all third-party actions and pin them to immutable releases Upgrade all third-party GitHub Actions and pin them to immutable releases May 28, 2026
@jeancochrane jeancochrane changed the title Upgrade all third-party GitHub Actions and pin them to immutable releases Upgrade all third-party GitHub Actions and pin them to commit hashes May 28, 2026
@jeancochrane jeancochrane marked this pull request as ready for review May 28, 2026 21:37
@jeancochrane jeancochrane removed their request for review May 29, 2026 15:39

@kyrasturgill kyrasturgill left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This at a high level makes sense to me! Given that the checks are passing with these changes, I have no concerns or flags. Thank you for proactively protecting us from bad actors!

@jeancochrane jeancochrane merged commit c482b24 into master Jun 1, 2026
9 checks passed
@jeancochrane jeancochrane deleted the copilot/upgrade-ccao-data-ptaxsim branch June 1, 2026 19:16
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