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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't modify the path when the path contains multiple slashes. #1289

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@ksullivan
Copy link

commented Aug 23, 2018

We have a node proxy (say "https://node.com") which points to a wordpress site (say "https://word.com").

If you go to "https://word.com///pricing", wordpress will redirect you to "https://word.com/pricing". Cool.

If you go to "https://node.com///pricing", the proxy only passes "/pricing" through to wordpress (not "///pricing"), so it doesn't redirect to "https://node.com/pricing" and you stay on "https://node.com///pricing". 馃槩

It would be nice if the proxy didn't interfere with the path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can鈥檛 perform that action at this time.