Yes, I think you're right.  I'll remove that so it just tests for the isInputCloseNotified and rerun the tests just to be sure.

Thanks for the feedback,

--Jamil

On 4/7/20 1:44 PM, Xuelei Fan wrote:
The conContext.isBroken condition in line 1124 is duplicated, and could be safely removed, I think.  Otherwise, looks good to me.  I don't think there's need for another round of RFR.

Thanks,
Xuelei

On 4/7/2020 12:53 PM, Jamil Nimeh wrote:
Hello all,

This is a fix that brings the JSSE client from JDK 11+ back in line with the JDK 8 behavior of delivering SSLException on subsequent reads after a handshake has failed due to an alert condition.

JBS: https://bugs.openjdk.java.net/browse/JDK-8242294

Webrev: https://cr.openjdk.java.net/~jnimeh/reviews/8242294/webrev.01

Thanks,

--Jamil

Reply via email to