Re: [13] RFR: 8216498: Confusing and unneeded wrapping of SSLHandshakeException

2019-01-11 Thread Chris Hegarty
On 11/01/2019 11:20, Daniel Fuchs wrote: Thanks Chris. More extensive test campaign revealed that I missed another wrapping point in HttpClientImpl::sendAsync. New webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8216498/webrev.01 Good catch. The updated webrev looks good. -Chris.

Re: [13] RFR: 8216498: Confusing and unneeded wrapping of SSLHandshakeException

2019-01-11 Thread Daniel Fuchs
Thanks Chris. More extensive test campaign revealed that I missed another wrapping point in HttpClientImpl::sendAsync. New webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8216498/webrev.01 best regards, -- daniel On 10/01/2019 20:18, Chris Hegarty wrote: On 10 Jan 2019, at 18:23, Daniel

Re: [13] RFR: 8216498: Confusing and unneeded wrapping of SSLHandshakeException

2019-01-10 Thread Chris Hegarty
> On 10 Jan 2019, at 18:23, Daniel Fuchs wrote: > > Hi, > > Please find below a fix for: > 8216498: Confusing and unneeded wrapping of SSLHandshakeException > https://bugs.openjdk.java.net/browse/JDK-8216498 > > Webrev: > http://cr.openjdk.java.net/~dfuchs/webrev_8216498/webrev.00/ This look

[13] RFR: 8216498: Confusing and unneeded wrapping of SSLHandshakeException

2019-01-10 Thread Daniel Fuchs
Hi, Please find below a fix for: 8216498: Confusing and unneeded wrapping of SSLHandshakeException https://bugs.openjdk.java.net/browse/JDK-8216498 Webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8216498/webrev.00/ When an IOException occurs in the java.net.http HTTP/1.1 stack it eventually g