Skip to content
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

Request parser strips leading slash from path #1512

Closed
YuppY opened this Issue May 12, 2017 · 2 comments

Comments

2 participants
@YuppY
Copy link
Contributor

YuppY commented May 12, 2017

I am faced with strange behavior of gunicorn: different requests GET /foo and GET //foo are parsed with same path /foo.
It is caused by fix of #297

Here is PR with fix of this issue: #1511
Could anyone review it?

@benoitc

This comment has been minimized.

Copy link
Owner

benoitc commented May 14, 2017

imo considering // as / is the correct behaviour and is in place since a long time.

Did it cause any issue on your side? How to reproduce it?

@YuppY

This comment has been minimized.

Copy link
Contributor Author

YuppY commented May 14, 2017

Replied with email.

YuppY pushed a commit to YuppY/gunicorn that referenced this issue Dec 10, 2017

YuppY pushed a commit to YuppY/gunicorn that referenced this issue Dec 10, 2017

YuppY pushed a commit to YuppY/gunicorn that referenced this issue Dec 16, 2017

berkerpeksag added a commit to berkerpeksag/gunicorn that referenced this issue Jan 11, 2018

fofanov pushed a commit to fofanov/gunicorn that referenced this issue Mar 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.