Re: RFR 8244652: Add test for non utf-8 response handling by websocket client

2020-05-14 Thread Rahul
Thanks Daniel I would proceed with the push if there are no further comments. On 13/05/2020, 12:39, "Daniel Fuchs" wrote: Hi Rahul, That looks good to me thanks! I am still teetering on whether we should also have altered the specification of WSHandshakeException, and

Re: RFR 8244652: Add test for non utf-8 response handling by websocket client

2020-05-13 Thread Daniel Fuchs
Hi Rahul, That looks good to me thanks! I am still teetering on whether we should also have altered the specification of WSHandshakeException, and explicitly changed the signature of WSHandshakeException::getResponse to return an HttpResponse instead of an HttpResponse. On the one hand - our im