Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2426#discussion_r163476600
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java
---
@@ -513,6 +530,17 @@ public void onPropertyModified(final
PropertyDescriptor descriptor, final String
results.add(new
ValidationResult.Builder().subject("Proxy").valid(false).explanation("If Proxy
username is set, proxy host must be set").build());
}
+ final boolean proxyTypeSet =
validationContext.getProperty(PROP_PROXY_TYPE).isSet();
+
+ if(proxyHostSet &&
!validationContext.getProperty(PROP_PROXY_TYPE).isSet()) {
--- End diff --
I believe this check could be removed based on the previous comment.
---
