[GitHub] [nifi] jfrazee commented on issue #4202: NIFI-7203 Support for TLS in Zookeeper Server

2020-04-13 Thread GitBox
jfrazee commented on issue #4202: NIFI-7203 Support for TLS in Zookeeper Server URL: https://github.com/apache/nifi/pull/4202#issuecomment-613075157 @natural Would you be able to snag the tests from #4207 and make the change to the logic and zookeeper.properties and then we'll just turn

[GitHub] [nifi] jfrazee commented on issue #4202: NIFI-7203 Support for TLS in Zookeeper Server

2020-04-13 Thread GitBox
jfrazee commented on issue #4202: NIFI-7203 Support for TLS in Zookeeper Server URL: https://github.com/apache/nifi/pull/4202#issuecomment-613039039 @natural Otherwise the implementation appears to be correct. This is an

[GitHub] [nifi] jfrazee commented on issue #4202: NIFI-7203 Support for TLS in Zookeeper Server

2020-04-13 Thread GitBox
jfrazee commented on issue #4202: NIFI-7203 Support for TLS in Zookeeper Server URL: https://github.com/apache/nifi/pull/4202#issuecomment-613029136 @natural Having also recently implemented this (#4207) I have a couple of comments: - Secure and insecure aren't mutually exclusive so if