[ https://issues.apache.org/jira/browse/NIFI-4790?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16337109#comment-16337109 ]
ASF GitHub Bot commented on NIFI-4790: -------------------------------------- Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/2426#discussion_r163476703 --- 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(); --- End diff -- Will always be true based on the previous comment. > Support HTTPS proxy in InvokeHTTP > --------------------------------- > > Key: NIFI-4790 > URL: https://issues.apache.org/jira/browse/NIFI-4790 > Project: Apache NiFi > Issue Type: Improvement > Reporter: Marco Gaido > Assignee: Marco Gaido > Priority: Major > > InvokeHTTP currently assumes that the proxy (if set) uses HTTP. We should > introduce a new property to indicate the protocol used by the proxy (either > http or https) in order to support also HTTPS proxies. -- This message was sent by Atlassian JIRA (v7.6.3#76005)