sequencescape
sequencescape copied to clipboard
🚨 [security] [js] Update vite 5.1.5 → 5.1.7 (patch)
🚨 Your current dependencies have known security vulnerabilities 🚨
This dependency update fixes known security vulnerabilities. Please see the details below and assess their impact carefully. We recommend to merge and deploy this as soon as possible!
Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.
What changed?
✳️ vite (5.1.5 → 5.1.7) · Repo · Changelog
Security Advisories 🚨
🚨 Vite's `server.fs.deny` did not deny requests for patterns with directories.
Summary
Vite dev server option
server.fs.deny
did not deny requests for patterns with directories. An example of such a pattern is/foo/**/*
.Impact
Only apps setting a custom
server.fs.deny
that includes a pattern with directories, and explicitly exposing the Vite dev server to the network (using--host
orserver.host
config option) are affected.Patches
Fixed in [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Details
server.fs.deny
uses picomatch with the config of{ matchBase: true }
. matchBase only matches the basename of the file, not the path due to a bug (micromatch/picomatch#89). The vite config docs read like you should be able to set fs.deny to glob with picomatch. Vite also does not set{ dot: true }
and that causes dotfiles not to be denied unless they are explicitly defined.Reproduction
Set fs.deny to
['**/.git/**']
and then curl for/.git/config
.
- with
matchBase: true
, you can get any file under.git/
(config, HEAD, etc).- with
matchBase: false
, you cannot get any file under.git/
(config, HEAD, etc).
Release Notes
5.1.7
Please refer to CHANGELOG.md for details.
5.1.6
Please refer to CHANGELOG.md for details.
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 19 commits:
release: v5.1.7
fix: `fs.deny` with globs with directories (#16250)
release: v5.1.6
chore(deps): update dependency @eslint-types/typescript-eslint to v7 (#16132)
chore(deps): update all non-major dependencies (#16131)
docs(static-deploy): update workflow (#15773)
fix: server middleware mode resolve (#16122)
fix: check for publicDir before checking if it is a parent directory (#16046)
docs: support using different sponsor images for dark mode
fix(esbuild): update tsconfck to fix bug that could cause a deadlock (#16124)
fix: handle function property extension in namespace import (#16113)
release: [email protected]
fix(plugin-legacy): respect modernTargets option if renderLegacyChunks disabled (#15789)
style(utils): remove null check (#16112)
fix(worker): throw error when circular worker import is detected and support self referencing worker (#16103)
docs: add reference link for set debug environment variable on Windows (#16097)
fix(worker): hide "The emitted file overwrites" warning if the content is same (#16094)
refactor(runtime): share more code between runtime and main bundle (#16063)
fix: escape single quote when relative base is used (#16060)
Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase
.
All Depfu comment commands
- @depfu rebase
- Rebases against your default branch and redoes this update
- @depfu recreate
- Recreates this PR, overwriting any edits that you've made to it
- @depfu merge
- Merges this PR once your tests are passing and conflicts are resolved
- @depfu cancel merge
- Cancels automatic merging of this PR
- @depfu close
- Closes this PR and deletes the branch
- @depfu reopen
- Restores the branch and reopens this PR (if it's closed)
- @depfu pause
- Ignores all future updates for this dependency and closes this PR
- @depfu pause [minor|major]
- Ignores all future minor/major updates for this dependency and closes this PR
- @depfu resume
- Future versions of this dependency will create PRs again (leaves this PR as is)
Code Climate has analyzed commit b677001e and detected 0 issues on this pull request.
The test coverage on the diff in this pull request is 100.0% (50% is the threshold).
This pull request will bring the total coverage in the repository to 86.7% (0.0% change).
View more on Code Climate.