Skip to content

[LOW] Pin actions/checkout to its reviewed commit - #3205

Open
OskarEichler wants to merge 1 commit into
iamkun:devfrom
OskarEichler:codex/security-pin-checkout
Open

OskarEichler wants to merge 1 commit into
iamkun:devfrom
OskarEichler:codex/security-pin-checkout

Conversation

@OskarEichler

Copy link
Copy Markdown

Summary

Both test and privileged release workflows referenced actions/checkout through the mutable v3 tag. If that tag were moved or the upstream action were compromised, unreviewed action code would execute in these workflows.

Fix

Pin both uses to the reviewed v3 commit a37ce9120846195fa4ece8f58b268e6043cb2f26 while retaining an inline v3 annotation for maintainability.

Validation

  • The SHA was resolved directly from the upstream action repository
  • Workflow YAML parsed successfully
  • Diff checks passed

Breaking changes

None.

@lbesecker195 lbesecker195 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verified: a37ce912 is exactly what the actions/checkout@v3 tag resolves to today. LGTM.

This branch has not been deployed

No deployments
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