users@jersey.java.net

[Jersey] Re: JERSEY-2469

From: Marek Potociar <marek.potociar_at_oracle.com>
Date: Tue, 8 Apr 2014 19:20:55 +0200

No need to resubmit. I can see you in the OCA list now.

Marek

On 07 Apr 2014, at 18:42, Ezequiel Mario <ezequielballesi_at_gmail.com> wrote:

> Great will do that.
> As for the OCA, I've submit a PR some time ago with the OCA. I can send it again if needed.
>
> Thanks!
> Ezequiel.
>
> On Apr 7, 2014 1:33 PM, "Marek Potociar" <marek.potociar_at_oracle.com> wrote:
> Hi Ezequiel,
>
> Feel free to submit a pull request but do not forget to provide a reproducer unit test that demonstrates the issue if the fix is not applied.
>
> Also, please do not forget to send us a signed OCA.
>
> Thanks,
> Marek
>
>
> On 03 Apr 2014, at 16:52, Ezequiel Mario <ezequielballesi_at_gmail.com> wrote:
>
>> Hi all!,
>>
>> I encountered an issue on UriBuilder implementation of jersey; it seems that is not following the spec for query parameter encoding as in https://jax-rs-spec.java.net/nonav/2.0/apidocs/javax/ws/rs/core/UriBuilder.html.
>>
>> Query parameters need to be encoded using "the rules of the application/x-www-form-urlencoded media type"; but it seems that the implementation is following the less strict encoding as defined in RFC 3986 : 3.4. Query; that allows more chars as part of a query param.
>>
>> I believe that the UriBuilder spec has a stricter encoding for query param due to some browser erroneous implementation;
>> "The characters slash ("/") and question mark ("?") may represent data
>> within the query component. Beware that some older, erroneous
>> implementations may not handle such data correctly when it is used as
>> the base URI for relative references (Section 5.1), apparently
>> because they fail to distinguish query data from path data when
>> looking for hierarchical separators."
>>
>> I filled an issue for this since I did not find one exactly as this one (https://java.net/jira/browse/JERSEY-2469).
>> Also I have a fix ready; and I can send a pull request.
>>
>> Any comments will be appreciated!
>> (Either if you feel this is a real issue or not)
>>
>> Keep on the excellent work!
>>
>> Regards,
>> Ezequiel.
>>
>