search

google / inverting-proxy

Reverse proxy that inverts the direction of traffic
Apache License 2.0
251 stars 73 forks source link

agent: Normalize URLs #19

Open JerryLeiDing opened 5 years ago

JerryLeiDing commented 5 years ago

We should normalize all URLs before sending requests.

Currently, an extra empty path segment in the URL can cause problems if the request method is rewritten.

ojarjur commented 5 years ago

Similarly, we should ensure that if the initial request to the proxy results in a redirect, that the method does not get changed to GET.

We know that in our use case we never want the method to be changed on redirects, so we should see if we can actively disable that behavior

ojarjur commented 5 years ago

I added https://github.com/google/inverting-proxy/issues/20 to track the related issue of detecting when the request method changes.