It may be not needed to update the handleException() method.

- 440            handleException(iioe);
+ 440            if (resumable) {
+                    throw iioe;
+                } else {
+                    handleException(iioe);
+                }

Otherwise, looks good to me.

Xuelei

On 6/4/2020 8:13 AM, Prasadrao Koppula wrote:
Hi,

Could you please review this patch. For timeout/interrupts, JDK11u+ releases, SSLSocket:getSession behavior is different, compare to JDK8u. i.e, connection is in open state for timeout/interrupts exception. For comparability reasons, this fix will close connection for getSession timeout/ interrupts.

Bug: https://bugs.openjdk.java.net/browse/JDK-8246031

Webrev: http://cr.openjdk.java.net/~pkoppula/8246031/webrev.00/

Thanks,

Prasad.K

Reply via email to