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 2022/05/30 08:19:37 UTC

[GitHub] [pulsar] lhotari commented on pull request #15824: Switch to rely on Netty for Hostname Verification

lhotari commented on PR #15824:
URL: https://github.com/apache/pulsar/pull/15824#issuecomment-1140850413

   There are 2 test failures:
   * org.apache.pulsar.client.api.AuthenticationTlsHostnameVerificationTest ► testTlsSyncProducerAndConsumerWithInvalidBrokerHost
   * org.apache.pulsar.proxy.server.ProxyWithAuthorizationTest ► 
   testTlsHostVerificationProxyToClient
   
   I debugged and fixed the first one. The test was invalid.
   I pushed the commit to my fork: https://github.com/lhotari/pulsar/commit/7c9545ec
   
   One observation is that when the handshake fails because of hostname verification, the client will keep on retrying for some time. Here's an example log: https://gist.github.com/lhotari/8455fc155e0c7526398f36e9bd0bd88c 
   (can be reproduced by running AuthenticationTlsHostnameVerificationTest with the above commit)
   
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org