pkelly-nifi commented on a change in pull request #5016:
URL: https://github.com/apache/nifi/pull/5016#discussion_r617670684
##########
File path:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java
##########
@@ -857,6 +859,9 @@ public void onTrigger(ProcessContext context,
ProcessSession session) throws Pro
statusAttributes.put(STATUS_MESSAGE, statusMessage);
statusAttributes.put(REQUEST_URL, url.toExternalForm());
statusAttributes.put(TRANSACTION_ID, txId.toString());
+ if(context.getProperty(PROP_FOLLOW_REDIRECTS).asBoolean()) {
Review comment:
Thank you for your feedback. I debated the same thing but went for it
this way to avoid duplication since if redirects are disabled, the "response"
url would always be identical to the "request" url. I'd be happy to always add
it as well. It might make some flows cleaner if they can rely on the
"response" attribute always being set. Let me submit that change.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
