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

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

onders86 opened a new pull request #2158: CAMEL-12097- handle if both path and 
query params set and supress wit…
URL: https://github.com/apache/camel/pull/2158
 
 
   …h the value in the path

----------------------------------------------------------------
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:
[email protected]


> Combination of path param and query param does not work
> -------------------------------------------------------
>
>                 Key: CAMEL-12097
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12097
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-restlet
>    Affects Versions: 2.20.1
>            Reporter: Radek Mensik
>             Fix For: 2.20.2, 2.21.0
>
>
> If in route are used both path parameter and query parameter:
> {code:java}
> restlet:{{data.access.url}}/v2/users/{userId}?projection=full
> {code}
> then because of commit :
> https://github.com/apache/camel/commit/4da8d0ad4472e69a5728f67fa121b3b4150fdc33#diff-470ee30498bbf9ea5b6ea223650c0e1c
> on lines 216-219 are escaped "{" "}" so userId in my example is never 
> replaced.
> Escaping is done in URISupport.appendParametersToURI on line:
> {code:java}
> URI uri = new URI(normalizeUri(originalURI));
> {code}
> There is workaround to use Exchange.HTTP_QUERY header for query parameter and 
> path must not contain query parameters.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to