Skip to content

ci: pin third-party actions to commit SHAs - #325

Merged
B4nan merged 1 commit into
masterfrom
chore/pin-github-actions
Aug 20, 2026
Merged

ci: pin third-party actions to commit SHAs#325
B4nan merged 1 commit into
masterfrom
chore/pin-github-actions

Conversation

@B4nan

@B4nan B4nan commented Aug 20, 2026

Copy link
Copy Markdown
Member

This pins every third-party action in the workflows to a full commit SHA, keeping the resolved version tag as a trailing comment. Renovate understands that convention and updates the SHA and comment together.

Same change as apify/crawlee#4051, rolled out team-wide. The trigger was the v11 tag of EndBug/add-and-commit moving to a broken release that failed to load and killed the crawlee publish workflow. With SHA pins, a tag moving under us, by accident or by compromise, can't break or hijack CI anymore. Where EndBug/add-and-commit is used, it's pinned to v11.0.0, the last working release.

Own-org references (apify/*) stay on floating refs on purpose, since we control those repos.

Floating tags can move to broken or malicious commits, as happened with
EndBug/add-and-commit v11 (see apify/crawlee#4051). Pin all third-party
actions to full commit SHAs with the resolved version tag in a trailing
comment. Own-org (apify/*) references stay on floating refs.
@B4nan B4nan added the t-tooling Issues with this label are in the ownership of the tooling team. label Aug 20, 2026
@github-actions github-actions Bot added this to the 147th sprint - Tooling team milestone Aug 20, 2026
@github-actions github-actions Bot added the tested Temporary label used only programatically for some analytics. label Aug 20, 2026
@B4nan
B4nan requested a review from barjin August 20, 2026 11:28
@B4nan
B4nan merged commit 2b3db76 into master Aug 20, 2026
4 checks passed
@B4nan
B4nan deleted the chore/pin-github-actions branch August 20, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants