exceptionfactory commented on code in PR #10151:
URL: https://github.com/apache/nifi/pull/10151#discussion_r2245470310
##########
nifi-extension-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java:
##########
@@ -542,7 +542,11 @@ public class InvokeHTTP extends AbstractProcessor {
public static final Relationship RESPONSE = new Relationship.Builder()
.name("Response")
- .description("Response FlowFiles transferred when receiving HTTP
responses with a status code between 200 and 299.")
+ .description("""
+ Response FlowFiles transferred when receiving HTTP responses with
a status code between 200 and 299.
+ Enabling [Response Generation Required] changes routing behavior
and sends FlowFiles to the Response relationship regardless of status code
received.
+ """
+ )
Review Comment:
Thanks for the additional recommendation, I agree that making it clearer is
needed. How about this updated version:
Enabling [Response Generation Required] changes routing behavior, sending
unsuccessful responses to their corresponding relationships and also sending
FlowFiles to the Response relationship as well, regardless of status code
received.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
