This repository has been archived by the owner on Apr 22, 2023. It is now read-only.
path: fix bugs related to paths with trailing slashes #4536
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix issue #4520
After look more closely at the
splitPath
codepath.basename
wasn't the only method there was (or would be) wrong.path.extname('/file.ext/')
returned''
(empty string). It now returns.ext
path.dirname('////')
returned//
, it now returns '/'.path.basename('/file.ext/')
returnedfile.ext/
, it now returnsfile.ext
.This should be tested on windows as well, I have a license but I have never managed to compile node on windows properly. Could someone else test this?
Note: originally I had hoped to simply fix the
splitPathRe
regexp, but matching a not-fixed number of/
at multiply positions did quickly become complicated./cc @bnoordhuis