Skip to content

Upgrade vite to update esbuild vulnerable version #137

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

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

anumol-baby
Copy link
Contributor

@anumol-baby anumol-baby commented Feb 27, 2025

This PR aims to fix the alert #34( https://github.com/github/codespaces-react/security/dependabot/34)
This closes issue https://github.com/github/vuln-mgmt/issues/138001
esbuild is introduced as a transitive dependency for the vite package.Hence we have tried upgrading the vite package to 6.20 from 4.5.9 .
Since this is a basic vite project we have ensured the below based on the migration guide :

  • The vite.config.js file content is using the ESM syntax.
  • The closest package.json file has "type": "module", or use the .mjs/.mts extension, e.g. vite.config.mjs or vite.config.mts.

Note: The codeql check analyze yml is running and failing as there is no source code to analyze even though it does not come under the scope of this repo.

@anumol-baby anumol-baby marked this pull request as ready for review February 27, 2025 18:37
@anpaskin anpaskin merged commit c7b120e into main Mar 4, 2025
2 of 3 checks passed
@anpaskin anpaskin deleted the releases/anu/fix-esbuild-vulnerability-alert branch March 4, 2025 22:12
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