Skip to content

fix: dynamic-import-vars plugin normalize path issue#16518

Merged
patak-cat merged 2 commits into
vitejs:mainfrom
godky:main
Apr 25, 2024
Merged

fix: dynamic-import-vars plugin normalize path issue#16518
patak-cat merged 2 commits into
vitejs:mainfrom
godky:main

Conversation

@godky
Copy link
Copy Markdown
Contributor

@godky godky commented Apr 24, 2024

Description

Parameters wrapped with ` may produce unexpected results in path.normalize or path.posix.normalize
For example:

path.posix.normalize('../../foo')     // '../../foo'
path.posix.normalize('`../../foo`')   // 'foo`'

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p3-minor-bug An edge case that only affects very specific usage (priority)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants