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/02/13 02:02:15 UTC

[GitHub] jai1 commented on issue #1225: Enable specification of TLS Protocol Versions and Cipher Suites

jai1 commented on issue #1225: Enable specification of TLS Protocol Versions and Cipher Suites
URL: https://github.com/apache/incubator-pulsar/pull/1225#issuecomment-365127017
 
 
   @rdhabalia - Have addressed your comments and added Tests
   @merlimat - Have created a ticket for adding documentation - will do it tomorrow or so 
   @maskit - Except for SecurityUtility.java  (line below) I didn't find any hardcoding - here too the code looks ok to me
   ```
   SSLContext sslCtx = SSLContext.getInstance("TLS");
   ```
   Can I get a +1 for this?
   

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