We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restify Version: master Node.js Version: 16.17.0
prePath middleware should reduce path /// to /.
///
/
prePath middleware reduces /// to `` (empty string).
Test case: master...mridgway:node-restify:prePathEmptyString
This code will parse an empty string which returns null for pathname. The router will then assert the pathname as a string and throw.
pathname
Yes, if/when I have some time.
The text was updated successfully, but these errors were encountered:
restify.pre.sanitizePath()
AssertionError
No branches or pull requests
Restify Version: master
Node.js Version: 16.17.0
Expected behaviour
prePath middleware should reduce path
///
to/
.Actual behaviour
prePath middleware reduces
///
to `` (empty string).Repro case
Test case: master...mridgway:node-restify:prePathEmptyString
Cause
This code will parse an empty string which returns null for
pathname
. The router will then assert thepathname
as a string and throw.Are you willing and able to fix this?
Yes, if/when I have some time.
The text was updated successfully, but these errors were encountered: