Skip to content

ci: pin actions to commit hashes#260

Open
kaze-cow wants to merge 1 commit intomainfrom
pin-actions-20260407152011
Open

ci: pin actions to commit hashes#260
kaze-cow wants to merge 1 commit intomainfrom
pin-actions-20260407152011

Conversation

@kaze-cow
Copy link
Copy Markdown
Contributor

@kaze-cow kaze-cow commented Apr 7, 2026

Summary

  • Replaces mutable action version tags (e.g. @v4) with locked commit SHAs
  • Tag names are preserved as inline comments for readability
  • Prevents supply chain attacks from compromised or force-pushed tags

Generated by pin-actions.sh

Replaces mutable version tags with locked commit SHAs to prevent
supply chain attacks from compromised or force-pushed tags.
@kaze-cow kaze-cow requested a review from a team as a code owner April 7, 2026 06:20
@kaze-cow kaze-cow self-assigned this Apr 7, 2026
@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedsinon@​15.0.310010010050100
Addedsolhint-plugin-prettier@​0.0.5971007479100
Addedts-node@​10.9.19610010082100
Addedsolidity-coverage@​0.8.29810010085100
Addedyargs@​17.7.19910010087100
Addedsolhint@​5.0.19810010087100
Addedtypescript@​5.0.41001009010090
Addedprettier@​2.8.7921009890100
Addedprettier-plugin-solidity@​1.1.310010010093100

View full report

@socket-security
Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Critical
Critical CVE: npm pbkdf2 returns predictable uninitialized/zero-filled memory for non-normalized or unimplemented algos

CVE: GHSA-h7cp-r72f-jxh6 pbkdf2 returns predictable uninitialized/zero-filled memory for non-normalized or unimplemented algos (CRITICAL)

Affected versions: >= 3.0.10 < 3.1.3

Patched version: 3.1.3

From: ?npm/hardhat@2.13.1npm/@safe-global/safe-core-sdk-types@2.2.0npm/@safe-global/protocol-kit@1.2.0npm/solidity-coverage@0.8.2npm/ethereum-waffle@3.4.4npm/pbkdf2@3.1.2

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/pbkdf2@3.1.2. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn Critical
Critical CVE: npm pbkdf2 silently disregards Uint8Array input, returning static keys

CVE: GHSA-v62p-rq8g-8h59 pbkdf2 silently disregards Uint8Array input, returning static keys (CRITICAL)

Affected versions: >= 1.0.0 < 3.1.3

Patched version: 3.1.3

From: ?npm/hardhat@2.13.1npm/@safe-global/safe-core-sdk-types@2.2.0npm/@safe-global/protocol-kit@1.2.0npm/solidity-coverage@0.8.2npm/ethereum-waffle@3.4.4npm/pbkdf2@3.1.2

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/pbkdf2@3.1.2. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn Critical
Critical CVE: npm sha.js is missing type checks leading to hash rewind and passing on crafted data

CVE: GHSA-95m3-7q98-8xr5 sha.js is missing type checks leading to hash rewind and passing on crafted data (CRITICAL)

Affected versions: < 2.4.12

Patched version: 2.4.12

From: ?npm/hardhat@2.13.1npm/@safe-global/safe-core-sdk-types@2.2.0npm/@safe-global/protocol-kit@1.2.0npm/solidity-coverage@0.8.2npm/ethereum-waffle@3.4.4npm/sha.js@2.4.11

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/sha.js@2.4.11. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Copy link
Copy Markdown
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

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

Approved. The hash matches the expected versions

Just a small comment. The publish.yml workflow pins to actions/checkout@v3 while all other workflows use @v4. This was already the case before this PR (not introduced by it), but could be bumped to v4 in a follow-up.

@anxolin anxolin mentioned this pull request Apr 13, 2026
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.

2 participants