[
https://issues.apache.org/jira/browse/NIFI-747?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14962656#comment-14962656
]
Joseph Witt commented on NIFI-747:
----------------------------------
Changes look good with the PR and Tony's addition. Two observations:
1) We should call the new property "URI" something else such as 'path' since it
is only part of the overall URI from the user perspective.
2) The comments are great. Recommend adding one to signal that the 'Path'
annotation which is required on the servlets must also start with a '/'. We
don't detect/defend that in the code where we use it so good to just flag that.
THanks!
> ListenHTTP should expose property for the URL path to accept data on
> --------------------------------------------------------------------
>
> Key: NIFI-747
> URL: https://issues.apache.org/jira/browse/NIFI-747
> Project: Apache NiFi
> Issue Type: Improvement
> Components: Extensions
> Reporter: Mark Payne
> Assignee: Tony Kurc
> Labels: beginner, newbie
> Fix For: 0.4.0
>
>
> When using ListenHTTP, you have to post to
> http(s)://<hostname>:<port>/contentListener
> The /contentListener part is often problematic, as it's often mistyped and
> really serves no purpose, since the processor starts an embedded web server
> with only a single servlet.
> We cannot remove the "/contentListener" path because we need to maintain
> backward compatibility. However, we can provide a new property to configure
> it, setting the default to "/contentListener". This way, when a user adds a
> ListenHTTP Processor, he/she can change the path to "/"
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
