[GitHub] nifi issue #3102: NIFI-5737: Removing need client auth property as cluster c...

2018-10-22 Thread jtstorck 
Github user jtstorck commented on the issue:

https://github.com/apache/nifi/pull/3102
  
Based on @markap14 giving a +1 on the JIRA 
(https://issues.apache.org/jira/browse/NIFI-5737) and @thenatog's testing, I'll 
merge this so that it will be part of NiFi 1.8.0 RC3.


---

[GitHub] nifi issue #3102: NIFI-5737: Removing need client auth property as cluster c...

2018-10-22 Thread thenatog 
Github user thenatog commented on the issue:

https://github.com/apache/nifi/pull/3102
  
+1

Reviewed the code changes. Most of the changes were related to tests. The 
needClientAuth attribute is not used all that much in actual code. Tested 
InvokeHTTP with trust/keystores set up, site-to-site and secure clustering. 
Also tested insecure site-to-site and clustering. Tested using flow registry.


---