You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2018/09/27 22:03:57 UTC

[GitHub] merlimat opened a new pull request #2670: Fixed HTTP redirects with proxy handler

merlimat opened a new pull request #2670: Fixed HTTP redirects with proxy handler
URL: https://github.com/apache/pulsar/pull/2670
 
 
   ### Motivation
   
   Fixes #2660 
   
   There is a problem with the Jetty proxy servlet configuration that it makes it to not transparently follow HTTP redirects from brokers. This results in clients being redirected to a broker hostname that they cannot reach.
   
   ### Modifications
   
   Completely override the `createHttpClient()` from `AbstractProxyServlet` in order to ensure that `RedirectProtocolHandler` is always in the list of handlers for the client.
   
   ### Result
   
   Proxy will internally handle 307 redirection and will send a 200 to client.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services