Skip to content

[Snyk] Security upgrade vite from 5.4.20 to 6.4.2#1019

Open
Dargon789 wants to merge 2 commits intomainfrom
snyk-fix-5cb4a4e445ba81a50ee0e9e6da51bd4f
Open

[Snyk] Security upgrade vite from 5.4.20 to 6.4.2#1019
Dargon789 wants to merge 2 commits intomainfrom
snyk-fix-5cb4a4e445ba81a50ee0e9e6da51bd4f

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

snyk-top-banner

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

Snyk changed the following file(s):

  • examples/react-ak-basic-sign-client/package.json
⚠️ Warning
Failed to update the pnpm-lock.yaml, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
medium severity Directory Traversal
SNYK-JS-VITE-15922213

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:

🦉 Directory Traversal

@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 vite dependency from version 5.4.20 to 6.4.2 in the examples/react-ak-basic-sign-client/package.json file. Feedback indicates that this security upgrade is incomplete because the root package.json still contains the vulnerable version and the lockfile has not been updated. Additionally, the reviewer noted that moving from version 5 to 6 is a major upgrade that may introduce breaking changes requiring further code adjustments.

"@types/react-dom": "19.1.9",
"@vitejs/plugin-react": "4.2.1",
"vite": "5.4.20"
"vite": "6.4.2"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

critical

This security upgrade is incomplete and requires further changes.

  1. Incomplete dependency update: The vite dependency is also present in the root package.json at the vulnerable version 5.4.20. To fully address the security vulnerability, you must also upgrade vite in the root package.json.

  2. Lockfile update required: As noted in the PR description, pnpm-lock.yaml has not been updated. After updating all vite dependencies, you must run pnpm install to regenerate the lockfile. Without this, the vulnerability might not be fixed in all environments.

  3. Major version upgrade: This is an upgrade from vite v5 to v6, which is a major version change and may introduce breaking changes. Please review the vite v6 release notes and migration guide to ensure compatibility with your project and update your code as needed.

@Dargon789 Dargon789 linked an issue Apr 9, 2026 that may be closed by this pull request
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.

[vc]: #main-auth

2 participants