markobean commented on code in PR #7639:
URL: https://github.com/apache/nifi/pull/7639#discussion_r1303579528
##########
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java:
##########
@@ -130,7 +130,8 @@
@Tags({"http", "https", "rest", "client"})
@InputRequirement(Requirement.INPUT_ALLOWED)
@CapabilityDescription("An HTTP client processor which can interact with a
configurable HTTP Endpoint. The destination URL and HTTP Method are
configurable."
- + " FlowFile attributes are converted to HTTP headers and the FlowFile
contents are included as the body of the request (if the HTTP Method is PUT,
POST or PATCH).")
+ + " When the HTTP Method is PUT, POST or PATCH, the FlowFile contents
are included as the body of the request and FlowFile attributes are converted"
+ + " to HTTP headers (optionally, if configured to be sent via the
'Attributes to Send' property)")
Review Comment:
Oops, I missed that I used `.name` and not `.displayName`. Good catch. Also,
I prefer to keep the work "optionally", but made the reference to generic
"configuration properties" as suggested.
--
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]
