Re: RFR 8240666: Websocket client’s OpeningHandshake discards the HTTP response body

[Daniel Fuchs]

Hi Rahul,

LGTM.
 111                 WebSocketHandshakeException wse = 
(WebSocketHandshakeException) t;
 112                 out.println("Status code is " + 
wse.getResponse().statusCode());
 113                 out.println("Response is " + 
wse.getResponse().body());
 114                 assertNotNull(wse.getResponse());


I'd suggest moving line 114 two lines up (just after line 111)
to avoid triggering a NPE if wse.getResponse() returns null.
No need for a new webrev if that's the only change.

best regards,

-- daniel

On 06/05/2020 16:04, Rahul wrote:
Hi Pavel,

Thank you for the comment, the webrev has been updated.

webrev : http://cr.openjdk.java.net/~ryadav/webrev_8240666/webrev.00/index.html

- rahul

On 06/05/2020, 14:26, "Pavel Rappo" <pavel.ra...@oracle.com> wrote:

     An assertion of the form
     
         assertEquals(true, ((String)wse.getResponse().body()).contains("404"));
     
     looks odd. I'd suggest using any of
     
         assertTrue(boolean condition)
         assertTrue(boolean condition, String message)
     
     -Pavel
     
     > On 6 May 2020, at 14:12, Rahul <rahul.r.ya...@oracle.com> wrote:
     >
     > http://cr.openjdk.java.net/~ryadav/webrev_8240666/webrev.00/index.html