[GitHub] [tomcat-native] jfclere commented on pull request #10: fix an issue similar to BZ 55707

[GitBox <gi...@apache.org>]

jfclere commented on pull request #10:
URL: https://github.com/apache/tomcat-native/pull/10#issuecomment-881501106


   Note the openssl doc from SSL_CTX_set_tlsext_servername_callback:
   "The functionality provided by the servername callback is mostly superseded by the ClientHello callback, which can be set using SSL_CTX_set_client_hello_cb(). However, even where the ClientHello callback is used, the servername callback is still necessary in order to acknowledge the servername requested by the client."
   so both call back are needed.
   Note also: SecureNioChannel.processSNI that explains how NIO/NIO2 don't need the call back.


-- 
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: dev-unsubscribe@tomcat.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org