[jira] [Commented] (CAMEL-11658) URISyntaxException in RestletProducer

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on CAMEL-11658:
----------------------------------------

onderson commented on issue #1938: CAMEL-11658: test for RestletProducer handling unencoded headers
URL: https://github.com/apache/camel/pull/1938#issuecomment-387052035
 
 
   @koscejev , what's the use case for providing unencoded uri in http query?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> URISyntaxException in RestletProducer
> -------------------------------------
>
>                 Key: CAMEL-11658
>                 URL: https://issues.apache.org/jira/browse/CAMEL-11658
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-restlet
>    Affects Versions: 2.19.0
>            Reporter: Anton Koscejev
>            Priority: Minor
>
> CAMEL-10392 introduced session/cookie handling for camel-restlet. However, this also introduced a bug where new URI is created from resourceUri before it's encoded.
> Before this change the uri would be safely encoded within restlet's Request constructor.
> See this code in RestletProducer:
> {code}
>             String resourceUri = buildUri(endpoint, exchange);
>             URI uri = new URI(resourceUri); // <- causes exception on unencoded uri
>             request = new Request(endpoint.getRestletMethod(), resourceUri); // <- will encode uri
>             binding.populateRestletRequestFromExchange(request, exchange);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)