[jira] [Commented] (CAMEL-5458) Jetty/HTTP Pathless Consumer matchOnUriPrefix=true Breaks Producer bridgeEndpoint=true

["Scott England-Sullivan (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CAMEL-5458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13420707#comment-13420707 ] 

Scott England-Sullivan commented on CAMEL-5458:
-----------------------------------------------

Hello,

I have uploaded a patch.  It contains the following:

Camel-HTTP: 
HttpHelper.createURL(Exchange exchange, HttpEndpoint endpoint) now uses only the Endpoints defined URI and ignores the headers as documented.

Camel-Jetty:
JettyContentExchange.getUrl() did not set the ":" after the getScheme() and before the "//".
Added JettyMatchOnUriPrefixBridgedRouteTest that covers all variations of matched and unmatched bridged endpoint URIs.

Please let me know if you have any comments.

Thanks,
Scott ES
http://fusesource.com


                
> Jetty/HTTP Pathless Consumer matchOnUriPrefix=true Breaks Producer bridgeEndpoint=true
> --------------------------------------------------------------------------------------
>
>                 Key: CAMEL-5458
>                 URL: https://issues.apache.org/jira/browse/CAMEL-5458
>             Project: Camel
>          Issue Type: Bug
>            Reporter: Scott England-Sullivan
>         Attachments: CAMEL-5458.patch
>
>
> When a Jetty/HTTP endpoint that has the matchOnUriPrefix=true option set and is configured to match any URI (no path is specified) the HttpHelper concatenates the consumer path to the producer endpoint creating an HttpOperationUnsupportedException.
> For example the following will process as expected and the HttpHelper class will create the http://localhost:10021/proxy/ping consumer URI:
> {code}
> from("jetty:http://localhost:10020/ping?matchOnUriPrefix=true")
>     .to("jetty:http://localhost:10021/proxy/ping?bridgeEndpoint=true");
> {code}
> While this code will create http://localhost:10021/proxy/ping/ping and create the exception (the only difference is there isn't a path on the consumer):
> {code}
> from("jetty:http://localhost:10020?matchOnUriPrefix=true")
>     .to("jetty:http://localhost:10021/proxy/ping?bridgeEndpoint=true");
> {code}
> Test case and patch will be uploaded shortly.
> Thanks,
> Scott ES
> http://fusesource.com

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira