-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
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
checkPathAgainstBase with basePath '/' #27
Comments
Would you be able to contribute this code yourself? This is a fully open source project and only thrives when the community feeds back into it. I can make the change, but it's your discovery and you should get the recognition for it. |
can you check with the latest version v0.0.47 ? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks your great job.
I found problem when Server path is no base path like this.
This will call checkPathAgainstBase with basePath '/'. And the checkPathAgainstBase trim basePath last slash here, but if basePath is '/', basePaths is to be empty string. It's unexpected behavior, right?
so I think the code should be fixed like this.
The text was updated successfully, but these errors were encountered: