[
https://issues.apache.org/jira/browse/CXF-6606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14906871#comment-14906871
]
ASF GitHub Bot commented on CXF-6606:
-------------------------------------
GitHub user tylerbrazier opened a pull request:
https://github.com/apache/cxf/pull/91
CXF-6606 Encoded characters in URI are decoded multiple times during …
…preprocess
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tylerbrazier/cxf master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cxf/pull/91.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 #91
----
commit 55860e078bd923b6e2ad85dbc410faad647c6998
Author: Tyler Brazier <[email protected]>
Date: 2015-09-24T19:26:30Z
CXF-6606 Encoded characters in URI are decoded multiple times during
preprocess
----
> Encoded characters in URI are decoded multiple times during preprocess
> ----------------------------------------------------------------------
>
> Key: CXF-6606
> URL: https://issues.apache.org/jira/browse/CXF-6606
> Project: CXF
> Issue Type: Bug
> Components: JAX-RS
> Affects Versions: 3.1.2
> Reporter: Tyler Brazier
>
> When given a url like {{/api/%255E/users.json}},
> {{RequestPreprocessor#preprocess}} first does {{handleExtensionMappings}},
> which creates a {{PathSegmentImpl}} with a single arg constructor, decoding
> the path. The path on {{Message}} is set to the decoded path with a call to
> {{updatePath}} if the {{.json}} extension was found within the extension
> mappings. Back in {{RequestPreprocessor#preprocess}}, the path returned will
> be decoded again within the call to {{UriInfoImpl#getPath}}.
> This causes the {{%255E}} in the path to be decoded the first time as
> {{%5E}}, then the second time as {{^}}.
> I believe this could be fixed by constructing {{PathSegment}} within
> {{handleLanguageMappings}} and {{handleExtensionMappings}} with a second
> {{false}} argument.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
