ottobackwards commented on a change in pull request #3956: Added NTLM
Authentication based on presence of NTLM Domain property
URL: https://github.com/apache/nifi/pull/3956#discussion_r363383565
##########
File path:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java
##########
@@ -113,9 +114,10 @@
import static org.apache.commons.lang3.StringUtils.trimToEmpty;
@SupportsBatching
-@Tags({"http", "https", "rest", "client"})
+@Tags({"http", "https", "rest", "client","ntlm"})
@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."
+ + " Also supports NTLM authentication to communicate with a sharepoint
webservice"
+ " 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).")
Review comment:
Unless NTLM _only_ works with sharepoint, there isn't a need to specify it
here. If you _do_ specify it, you might want to capitalize it correctly.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
