Skip to content

fix(examples): bump vite to ^7.3.3 to fix security vulnerabilities#10324

Merged
chrisgervang merged 1 commit into
masterfrom
chr/fix-npm-security-basemap-browser
May 22, 2026
Merged

fix(examples): bump vite to ^7.3.3 to fix security vulnerabilities#10324
chrisgervang merged 1 commit into
masterfrom
chr/fix-npm-security-basemap-browser

Conversation

@chrisgervang
Copy link
Copy Markdown
Collaborator

@chrisgervang chrisgervang commented May 22, 2026

Background

Vite versions < 7.3.3 have two high-severity CVEs:

All 62 examples specified "vite": "^7.3.1" which allows installation of vulnerable versions.

Change List

  • Bump vite minimum version from ^7.3.1 to ^7.3.3 across all 62 example package.json files

Note

Low Risk
Low risk: this is a devDependency-only patch bump across example apps, with no runtime/library code changes; main risk is potential example build/dev-server behavior changes from the newer Vite patch.

Overview
Updates all example projects to require vite ^7.3.3 (from ^7.3.1) in devDependencies, ensuring installs pick up the patched Vite release that addresses known security issues in older versions.

Reviewed by Cursor Bugbot for commit c5dcffa. Bugbot is set up for automated code reviews on this repo. Configure here.

…abilities

Vite versions < 7.3.3 have two high-severity CVEs:
- GHSA-4w7w-66w2-5vf9 (Path Traversal in Optimized Deps .map Handling)
- GHSA-p9ff-h696-f583 (Arbitrary File Read via Dev Server WebSocket)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@chrisgervang chrisgervang requested a review from ibgreen May 22, 2026 19:39
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 83.37% (+0.004%) from 83.366% — chr/fix-npm-security-basemap-browser into master

Copy link
Copy Markdown
Collaborator

@ibgreen ibgreen left a comment

Choose a reason for hiding this comment

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

Nice. (We could just go all the way to 8.)

@chrisgervang
Copy link
Copy Markdown
Collaborator Author

Good point - I'm mainly trying to unblock your ability to run basemap-browser for testing #10228. I can look at 8 in a follow up in case there are breaking changes

@chrisgervang chrisgervang merged commit 34a5e71 into master May 22, 2026
6 checks passed
@chrisgervang chrisgervang deleted the chr/fix-npm-security-basemap-browser branch May 22, 2026 21:42
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.

3 participants