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

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

koscejev commented on issue #1938: CAMEL-11658: test for RestletProducer 
handling unencoded headers
URL: https://github.com/apache/camel/pull/1938#issuecomment-387057433
 
 
   @onderson, perhaps I used incorrect test class for this, but this is about 
using unencoded header values as path param, not query param. The test already 
does this for `username` header used as path param, so I simply reused this.
   
   Normally Camel internals are protocol-agnostic, so I would expect it will 
encode/decode header as required by the protocol, not just fail due to some 
values.

----------------------------------------------------------------
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:
us...@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)

Reply via email to