Node 5.7.1 path.normalize broken #5585
Labels
Comments
|
cc: @mscdex |
Merged
MylesBorins
pushed a commit
that referenced
this issue
Mar 7, 2016
Fixes a regression introduced by b212be0. path.normalize(''/a/b/c/../../../x/y/z'') should return '/x/y/z'. Fixes: #5585 PR-URL: #5589 Reviewed-By: Myles Borins <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Brian White <[email protected]>
|
I've added this commit to v5.7.2-proposal |
Fishrock123
pushed a commit
that referenced
this issue
Mar 8, 2016
Fixes a regression introduced by b212be0. path.normalize(''/a/b/c/../../../x/y/z'') should return '/x/y/z'. Fixes: #5585 PR-URL: #5589 Reviewed-By: Myles Borins <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Brian White <[email protected]>
Fishrock123
pushed a commit
that referenced
this issue
Mar 8, 2016
Fixes a regression introduced by b212be0. path.normalize(''/a/b/c/../../../x/y/z'') should return '/x/y/z'. Fixes: #5585 PR-URL: #5589 Reviewed-By: Myles Borins <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Brian White <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The following sample works fine in node v5.1.0 but is broken in node v5.7.1:
path.normalize("/a/b/c/../../../x/y/z")Correct (node v5.1.0):
/x/y/zBroken (node v5.7.1):
/a/x/y/zNote this works fine if the first path contains more than 1 character (e.g.
"/aa/b/c/../../../x/y/z")The text was updated successfully, but these errors were encountered: