Skip to content

fix: allow symlinked dev server roots#22539

Open
popsiclelmlm wants to merge 1 commit into
vitejs:mainfrom
popsiclelmlm:fix/symlink-fs-allow
Open

fix: allow symlinked dev server roots#22539
popsiclelmlm wants to merge 1 commit into
vitejs:mainfrom
popsiclelmlm:fix/symlink-fs-allow

Conversation

@popsiclelmlm
Copy link
Copy Markdown

Summary

  • include realpath counterparts for server.fs.allow directories so symlinked project roots can load files resolved through their target path
  • add coverage for symlinked root module loading and explicit fs.allow symlink entries

Fixes #16257

Tests

  • pnpm --filter vite build-bundle
  • pnpm exec vitest run packages/vite/src/node/server/tests/server-options.spec.ts
  • pnpm --filter vite exec oxfmt --check src/node/server/index.ts src/node/server/tests/server-options.spec.ts
  • pnpm --filter vite exec eslint src/node/server/index.ts src/node/server/tests/server-options.spec.ts
  • pnpm --filter vite build-types
  • pnpm --filter vite typecheck
  • git diff --check

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.

Pre-transform error when path contains a symlink

1 participant