Skip to content

[Snyk] Security upgrade @actions/core from 1.10.1 to 2.0.0#83

Open
revan-zhang wants to merge 2 commits intomainfrom
snyk-fix-9c1f7b09c18efd9bb995e8a7df63fd8a
Open

[Snyk] Security upgrade @actions/core from 1.10.1 to 2.0.0#83
revan-zhang wants to merge 2 commits intomainfrom
snyk-fix-9c1f7b09c18efd9bb995e8a7df63fd8a

Conversation

@revan-zhang
Copy link
Collaborator

@revan-zhang revan-zhang commented Mar 15, 2026

snyk-top-banner

Snyk has created this PR to fix 4 vulnerabilities in the yarn dependencies of this project.

Snyk changed the following file(s):

  • slack-notify-webhook/package.json
  • slack-notify-webhook/yarn.lock

Note for zero-installs users

If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

Vulnerabilities that will be fixed with an upgrade:

Issue Score
high severity Improper Handling of Highly Compressed Data (Data Amplification)
SNYK-JS-UNDICI-15518068
  710  
high severity Uncaught Exception
SNYK-JS-UNDICI-15518070
  710  
medium severity HTTP Request Smuggling
SNYK-JS-UNDICI-15518061
  620  
medium severity CRLF Injection
SNYK-JS-UNDICI-15518072
  455  

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Uncaught Exception
🦉 CRLF Injection


Open with Devin

Note

Medium Risk
Medium risk because this is a major-version upgrade of @actions/core that changes transitive dependencies (@actions/http-client, undici) and could affect runtime behavior of the GitHub Action despite no code changes.

Overview
Upgrades the GitHub Action dependency @actions/core from ^1.10.1 to ^2.0.0.

Refreshes yarn.lock accordingly, introducing new transitive packages (e.g., @actions/exec, @actions/io) and bumping @actions/http-client/undici to newer major versions to remediate security issues.

Written by Cursor Bugbot for commit f660233. This will update automatically on new commits. Configure here.

@revan-zhang
Copy link
Collaborator Author

revan-zhang commented Mar 15, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@socket-security
Copy link

socket-security bot commented Mar 15, 2026

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedaxios@​1.6.7986010096100
Added@​vercel/​ncc@​0.34.09910010082100
Added@​actions/​core@​2.0.39910010090100
Addedneedle@​2.9.19910010090100

View full report

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View 3 additional findings in Devin Review.

Open in Devin Review

},
"dependencies": {
"@actions/core": "^1.10.1",
"@actions/core": "^2.0.0",

Choose a reason for hiding this comment

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

🔴 dist/index.js not rebuilt after @actions/core major version bump — security fix has no effect

The package.json bumps @actions/core from ^1.10.1 to ^2.0.0 (intended as a vulnerability fix), but the bundled dist/index.js was not rebuilt. Since action.yml:30 specifies main: 'dist/index.js', GitHub Actions runs the bundled file, which still contains the old @actions/core v1.x code. The vulnerability fix therefore has no runtime effect — the action continues to use the old, vulnerable dependencies (undici@^5.25.4, @actions/http-client@^2.0.1). Additionally, slack-notify-webhook is not listed in the repo's build.sh, so automated builds won't pick it up either.

Prompt for agents
After updating the dependency in slack-notify-webhook/package.json, you need to rebuild the dist bundle so the fix actually takes effect at runtime. Run the following from the slack-notify-webhook directory:

yarn install
yarn build

This executes `ncc build src/index.js -m -o ./dist/` (as defined in package.json scripts) and produces an updated dist/index.js that bundles the new @actions/core v2 and its transitive dependencies.

Also consider adding slack-notify-webhook to the repo-level build.sh so future builds are automated. The build.sh at the repo root currently does not include slack-notify-webhook in its array of folders to build.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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