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 2020/04/30 17:59:22 UTC

[GitHub] [pulsar] rdhabalia commented on pull request #6853: Add Tls with keystore type config to support multi CAs

rdhabalia commented on pull request #6853:
URL: https://github.com/apache/pulsar/pull/6853#issuecomment-622010332


   I haven't gone through with the entire change but it seems it touches many different files and I think supporting multiple CA trust stores can be done in an easier way and might not need this big change.
   we can always bundle multiple CA trust store in one file and it can be provided to pulsar-broker. We also have similar kind of requirement and we are going to bundle multiple certs into one to support multiple ca usecase.
   https://alvinalexander.com/java/java-using-keytool-import-certificate-keystore/
   
   
   @merlimat can you also please review this PR.


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