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 2019/03/20 08:09:19 UTC

[GitHub] [pulsar] merlimat opened a new pull request #3869: Fixed issue with Authorization header missing after client gets redirected

merlimat opened a new pull request #3869: Fixed issue with Authorization header missing after client gets redirected
URL: https://github.com/apache/pulsar/pull/3869
 
 
   ### Motivation
   
   Jersey client uses by default the standard `HttpUrlConnection` from JDK. One issues that has been found is that, in newer JDK 8 versions (eg: 1.8.0_181), the `Authorization` header is not being included in the HTTP request after a redirect. The request than will fail with auth error.
   
   This is a problem for Token based authentication where we're using that header. 
   
   The authorization after redirection used to work at least until JDK 1.8.0_121 and there seems to be no way to configure the same behavior.
   
   ### Modifications
   
   Use Jersey JDK provider (based on Java NIO) instead of `HttpUrlConnectionProvider`. This is currently working correctly.
   
   Avoided for now the Netty provider since it's not supporting following the redirections.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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