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

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

GitHub user onders86 opened a pull request:

    https://github.com/apache/camel/pull/2158

    CAMEL-12097- handle if both path and query params set and supress wit…

    …h the value in the path

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/onders86/camel CAMEL-12097

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/2158.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2158
    
----
commit 0464c5e640c75c9a3bb6abe023e3d907856fde89
Author: onders86 <ondersezgin@...>
Date:   2018-01-02T15:12:44Z

    CAMEL-12097- handle if both path and query params set and supress with the 
value in the path

----


> 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