exceptionfactory commented on code in PR #8458:
URL: https://github.com/apache/nifi/pull/8458#discussion_r1513778626
##########
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:
Thanks for the additional explanation. So this should restore the previous
behavior prior to the multiple iterations that resulted from the need to move
away from `java.net.URL` construction.
With this background, it would be helpful to add or update a unit test that
evaluates the Transit URI on a Provenance Event, as one way to confirm the
expected behavior.
--
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
