Skip to content

fix: path resolve edge#19486

Open
OnlyWick wants to merge 1 commit intovitejs:mainfrom
OnlyWick:fix-path-resolve-edge
Open

fix: path resolve edge#19486
OnlyWick wants to merge 1 commit intovitejs:mainfrom
OnlyWick:fix-path-resolve-edge

Conversation

@OnlyWick
Copy link
Copy Markdown
Contributor

Description

close #19406.

path.posix.normalize also has issues with path resolution errors, so it also needs to be fixed.

@OnlyWick
Copy link
Copy Markdown
Contributor Author

Rerunning the CI, tests will pass

@OnlyWick
Copy link
Copy Markdown
Contributor Author

This is NOT just a relative path issue; this problem occurs with all uses of path.resolve. Perhaps we need to encapsulate path.resolve to avoid these issues.

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.

./x.js?foo=/../y.js is resolved as ./y.js instead of ./x.js

1 participant