exceptionfactory commented on code in PR #8458:
URL: https://github.com/apache/nifi/pull/8458#discussion_r1513612159
##########
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java:
##########
@@ -1012,7 +1013,7 @@ public void onTrigger(ProcessContext context,
ProcessSession session) throws Pro
}
}
- private Request configureRequest(final ProcessContext context, final
ProcessSession session, final FlowFile requestFlowFile, URL url) {
+ private Request configureRequest(final ProcessContext context, final
ProcessSession session, final FlowFile requestFlowFile, String url) {
Review Comment:
For clarification, passing a `URL` results in double-encoding, whereas
passing the raw `String` property does not? How was this working before the
implementation of the custom UriUtils?
--
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: issues-unsubscr...@nifi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
