Rahul,

Won't that start retaining the URL fragment? From 
https://tools.ietf.org/html/rfc6455#section-3

   Fragment identifiers are meaningless in the context of WebSocket URIs
   and MUST NOT be used on these URIs.  As with any URI scheme, the
   character "#", when not indicating the start of a fragment, MUST be
   escaped as %23.

-Pavel

> On 26 Jun 2020, at 13:03, Rahul Yadav <rahul.r.ya...@oracle.com> wrote:
> 
> Hello,
> 
> Request to have my fix reviewed for issue:
> 
> JDK-8245245  :  WebSocket can loose the URL encoding of URI query parameters
> 
> The fix updates the jdk.internal.net.http.websocket.OpeningHandshake
> to ensure that the URL is not reencoded/decoded and loose the original
> encoding
> 
> Issue:  https://bugs.openjdk.java.net/browse/JDK-8245245
> webrev: http://cr.openjdk.java.net/~ryadav/webrev_8245245/index.html
> 
> - rahul

Reply via email to