Skip to content

[Snyk] Security upgrade happy-dom from 17.1.0 to 20.8.9#1013

Open
Dargon789 wants to merge 1 commit intomainfrom
snyk-fix-d14032a0247f26bc6f52397fc7da902a
Open

[Snyk] Security upgrade happy-dom from 17.1.0 to 20.8.9#1013
Dargon789 wants to merge 1 commit intomainfrom
snyk-fix-d14032a0247f26bc6f52397fc7da902a

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

snyk-top-banner

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

Snyk changed the following file(s):

  • packages/siwx/package.json
⚠️ Warning
Failed to update the pnpm-lock.yaml, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
critical severity Arbitrary Code Injection
SNYK-JS-HAPPYDOM-13535083
critical severity Prototype Pollution
SNYK-JS-HAPPYDOM-13559227
high severity Arbitrary Code Injection
SNYK-JS-HAPPYDOM-15790746
high severity Insertion of Sensitive Information Into Sent Data
SNYK-JS-HAPPYDOM-15810980

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:

🦉 Arbitrary Code Injection
🦉 Prototype Pollution

@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Apr 9, 2026

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

Deployment failed with the following error:

Too many requests - try again in 5 minutes (more than 60, code: "api-deployments-flood").

Learn More: https://vercel.com/docs/limits#rate-limits

@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Apr 9, 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

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

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the happy-dom development dependency to version 20.8.9 in the packages/siwx package. Feedback indicates that the pnpm-lock.yaml file was not updated alongside this change, which is required to ensure consistent installations and prevent CI failures.

"@types/react": "19.1.15",
"@vitest/coverage-v8": "2.1.9",
"happy-dom": "17.1.0",
"happy-dom": "20.8.9",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

security-high high

The pnpm-lock.yaml file was not updated in this pull request. In a pnpm workspace, the lockfile must be synchronized with package.json to ensure that the specified version (and its security fixes) are actually installed. Merging this change without an updated lockfile will cause CI failures in environments using --frozen-lockfile and will prevent the security vulnerabilities from being correctly remediated in deployed environments.

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