Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hotfix: pin tj-actions/changed-files by SHA #28171

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

goxberry
Copy link

What does this PR do? What is the motivation?

This PR pins the tj-actions/changed-files GitHub action to a SHA corresponding to a version that is not compromised. For details on the compromise, see:

Compromised versions leak CI secrets, so this repo's CI secrets will need to be rotated.

Merge instructions

Merge readiness:

  • Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

Sorry, something went wrong.

The `tj-actions/changed-files` GitHub action has been compromised. For
details, see:

- https://semgrep.dev/blog/2025/popular-github-action-tj-actionschanged-files-is-compromised/
- https://www.stepsecurity.io/blog/harden-runner-detection-tj-actions-changed-files-action-is-compromised

Using this action without SHA pinning will leak CI secrets, so this
commit pins the action to a known good SHA corresponding to v45.0.8.

Signed-off-by: Geoffrey M. Oxberry <[email protected]>
@goxberry goxberry requested a review from a team as a code owner March 15, 2025 07:51
@github-actions github-actions bot added the Github Related to Github configurations label Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Github Related to Github configurations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant