You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2021/11/19 18:41:10 UTC

[GitHub] [zookeeper] MathewManu edited a comment on pull request #680: ZOOKEEPER-3174: Quorum TLS - support reloading trust/key store

MathewManu edited a comment on pull request #680:
URL: https://github.com/apache/zookeeper/pull/680#issuecomment-974313225


   @ivmaykov Appreciate commiting this PR. 
   I have a question regarding `ClientX509Util` reload, Is this implemented already or any plans? I see your comment in the checkin-
   > Reloading of key and trust store for ClientX509Util is not in this PR but could be added easily.
   
   I need the `ClientX509Util` reload functionality for an mTLS setup that I'm doing.
   Looking at the source, I could see that, `ClientX509Util extends X509Util`, and the reload functionality is implemented in the base class X509Util. 
   Does that mean, I can just add a new client config property (say `sslReloadCertFiles`) and call `enableCertFileReloading()` if it's `true` ?
   
   


-- 
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: notifications-unsubscribe@zookeeper.apache.org

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