Skip to content

[StepSecurity] ci: Harden GitHub Actions#25

Merged
cpanato merged 1 commit intochainguard-dev:mainfrom
step-security-bot:stepsecurity_remediation_1742514553
Mar 21, 2025
Merged

[StepSecurity] ci: Harden GitHub Actions#25
cpanato merged 1 commit intochainguard-dev:mainfrom
step-security-bot:stepsecurity_remediation_1742514553

Conversation

@step-security-bot
Copy link
Copy Markdown
Contributor

Summary

This pull request is created by StepSecurity at the request of @eslerm. Please merge the Pull Request to incorporate the requested changes. Please tag @eslerm on your message if you have any questions related to the PR.

Security Fixes

Pinned Dependencies

Ref: chainguard-dev/prodsec#60

Supersedes: #24

GitHub Action tags and Docker tags are mutable. This poses a security risk. GitHub's Security Hardening guide recommends pinning actions to full length commit.

Feedback

For bug reports, feature requests, and general feedback; please email support@stepsecurity.io. To create such PRs, please visit https://app.stepsecurity.io/securerepo.

Signed-off-by: StepSecurity Bot bot@stepsecurity.io

Signed-off-by: StepSecurity Bot <bot@stepsecurity.io>
@eslerm eslerm requested a review from cpanato March 21, 2025 00:06
path: ./src/github.com/chainguard-dev/exitdir

- uses: actions/cache@v3
- uses: actions/cache@2f8e54208210a422b2efd51efaa6bd6d7ca8920f # v3.4.3
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this should use the latest release

https://github.com/actions/cache/releases/tag/v4.2.3

${{ runner.os }}-go-${{ matrix.go-version }}-

- uses: ko-build/setup-ko@v0.6
- uses: ko-build/setup-ko@ace48d793556083a76f1e3e6068850c1f4a369aa # v0.6
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@eslerm
Copy link
Copy Markdown

eslerm commented Mar 21, 2025

I haven't reviewed specific actions or their minor change logs. In https://github.com/chainguard-dev/prodsec/issues/60 ProdSec is working to remove all mutable tags and to replace them with pin full-length commit SHAs to protect Chainguard from a future tj-actions like attack.

@cpanato
Copy link
Copy Markdown
Member

cpanato commented Mar 21, 2025

force merging, the failures are not related, we need to fix this job

@cpanato cpanato merged commit 358a9a5 into chainguard-dev:main Mar 21, 2025
4 of 6 checks passed
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