[jira] [Comment Edited] (CXF-7241) JAX-RS ContainerRequestContext#setRequestUri() will cause a 404 when Uri contains a query string

["Sergey Beryozkin (JIRA)" <ji...@apache.org>]

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

Sergey Beryozkin edited comment on CXF-7241 at 2/3/17 6:12 PM:
---------------------------------------------------------------

Thanks for the patch, I've applied it with the minor updates - the same code should work with another setRequestUri overload, the raw values should be set to preserve the encoded chars if any for JAX-RS Encoded annotations to work, and I only resetting a query if the new one is not null - it is not clear yet if resetting the existing queries with a possible null is expected
thanks


was (Author: sergey_beryozkin):
Thanks for the patch, I've applied it minor updates - the same code should work with another setRequestUri overload, the raw values should be set to preserve the encoded chars if any for JAX-RS Encoded annotations to work, and I only resetting a query if the new one is not null - it is not clear yet if resetting the existing queries with a possible null is expected
thanks

> JAX-RS ContainerRequestContext#setRequestUri() will cause a 404 when Uri contains a query string
> ------------------------------------------------------------------------------------------------
>
>                 Key: CXF-7241
>                 URL: https://issues.apache.org/jira/browse/CXF-7241
>             Project: CXF
>          Issue Type: Bug
>          Components: JAX-RS
>    Affects Versions: 3.1.9
>            Reporter: Ezequiel Rosas Garcia
>             Fix For: 3.2.0, 3.1.11
>
>
> Hi all.
> I have a PreMatching filter that attempts to set query parameters using #setRequestUri(). However, doing this causes CXF to always return 404 and never set the new query parameters.
> Then only exception to this was when the original URI both had no query parameters and ended without an slash. 
> I have a little test to show what I mean:
> https://github.com/nhtzr/osgiee-web/blob/master/src/test/java/mx/nhtzr/osgiee/web/internal/MyFilterTest.java
> I also have a proposed fix:
> https://github.com/apache/cxf/pull/234



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)