joewitt commented on code in PR #10151:
URL: https://github.com/apache/nifi/pull/10151#discussion_r2245453718
##########
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:
Recommend making this behavior extremely clear:
Response FlowFiles transferred when receiving HTTP responses with a status
code between 200 and 299.
Enabling [Response Generation Required] changes routing behavior and while
responses are routed to their proper relationships as defined a FlowFile is
also always routed 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]
