[ 
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)

Reply via email to