stoty commented on code in PR #2270: URL: https://github.com/apache/zookeeper/pull/2270#discussion_r2212244622
########## zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md: ########## @@ -1776,6 +1776,16 @@ and [SASL authentication for ZooKeeper](https://cwiki.apache.org/confluence/disp Specifies whether Online Certificate Status Protocol is enabled in client and quorum TLS protocols. Default: false +* *ssl.tcnative.ocsp* and *ssl.quorum.tcnative.ocsp* : Review Comment: I want to move away from directly setting JDK global system properties, not add more. I would perhaps make sense the server which does not have other code running in it, but definitely not for the client. The ZK TLS config code is currently not structured to handle server-only settings well, adding server-only settings could also confuse users. In any case, we agreed to try and keep the individual changes as small as possible, so this is outside the scope of this particular ticket. -- 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-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
