Skip to content

ci: pin github actions to pull commit SHAs#57

Merged
walkowif merged 1 commit into
insightsengineering:mainfrom
dzeveloper:pin-used-actions
Mar 31, 2026
Merged

ci: pin github actions to pull commit SHAs#57
walkowif merged 1 commit into
insightsengineering:mainfrom
dzeveloper:pin-used-actions

Conversation

@dzeveloper

Copy link
Copy Markdown
Contributor

This PR updates the GitHub Action workflows to use full length commit SHAs instead of mutable tags (e.g., @v2).
By pinning to a specific hash, we ensure that the code running in our CI/CD pipeline is exactly what we audited, protecting us from "tag shifting" where a version tag is moved to a different (potentially malicious) commit.

Using version tags like @V3 relies on trust that the maintainer’s account won't be compromised. If an attacker gains access to a dependency we use, they can push a malicious update to that tag, and our pipeline would automatically pull the compromised code.

Reference Incident: aquasecurity/trivy-action#541

@dzeveloper dzeveloper requested a review from a team as a code owner March 26, 2026 14:35
@github-actions

github-actions Bot commented Mar 26, 2026

Copy link
Copy Markdown

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@dzeveloper

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@walkowif walkowif left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@walkowif walkowif merged commit f3f8cdb into insightsengineering:main Mar 31, 2026
1 of 2 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 31, 2026
@walkowif

Copy link
Copy Markdown
Contributor

@dzeveloper Thanks for the PR! It's included in v3.0.3.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants