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

Replace unpinned actions with pinned action #3321

Merged
merged 1 commit into from
May 13, 2024

Conversation

stacklokbot
Copy link
Contributor

This is a Minder automated pull request.

This pull request replaces references to actions by tag to references to actions by SHA.

Verifies that any actions use pinned tags
Pinning an action to a full length commit SHA is currently the only way to use
an action as an immutable release. Pinning to a particular SHA helps mitigate
the risk of a bad actor adding a backdoor to the action's repository, as they
would need to generate a SHA-1 collision for a valid Git object payload.
When selecting a SHA, you should verify it is from the action's repository
and not a repository fork.

For more information, see
https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#using-third-party-actions

@stacklokbot stacklokbot requested a review from a team as a code owner May 13, 2024 17:15
Copy link
Contributor Author

@stacklokbot stacklokbot left a comment

Choose a reason for hiding this comment

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

✅ No Invisible Unicode Characters Detected.

@coveralls
Copy link

Coverage Status

coverage: 49.291% (+0.01%) from 49.28%
when pulling 8e5a228 on minder_replace_unpinned_actions_with_pinned_action
into 5183c10 on main.

@dmjb dmjb merged commit 4a8169d into main May 13, 2024
22 checks passed
@dmjb dmjb deleted the minder_replace_unpinned_actions_with_pinned_action branch May 13, 2024 17:31
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