-
-
Notifications
You must be signed in to change notification settings - Fork 11
chore: update deps, fix security vulns, upgrade deprecated CI actions #171
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -134,7 +134,7 @@ jobs: | |
|
|
||
| - name: 🔐 Log in to Docker Hub | ||
| if: github.event_name == 'release' || inputs.push | ||
| uses: docker/login-action@v3 | ||
| uses: docker/login-action@v4 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🚨 suggestion (security): Apply the same version-pinning strategy to As with the CodeQL steps, Suggested implementation:
|
||
| with: | ||
| registry: ${{ env.REGISTRY }} | ||
| username: ${{ secrets.DOCKER_USERNAME }} | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚨 suggestion (security): Consider pinning GitHub Actions to a specific commit SHA instead of a major tag for better supply-chain security.
Since you’re already updating CodeQL, it’s a good time to pin these to immutable SHAs (e.g.
github/codeql-action/*here andupload-sarifinmain-ci.yml). This improves build reproducibility and reduces the impact of any compromised or breaking upstream release; you can add a comment with the corresponding release tag for traceability.Suggested implementation:
github/codeql-actionv4 release (e.g. v4.0.0) and replace<CODEQL_ACTION_COMMIT_SHA>with that immutable SHA in all three steps to keep them in sync..github/workflows/main-ci.yml, locate anyupload-sarifusage (e.g.github/codeql-action/upload-sarif@v4or similar) and:github/codeql-action/upload-sarif@v4.0.0) for traceability.