Hi Prasad,

Change looks good to me, can you please put some comment why we are not performing "ssl handshake" if connectTime <= 0. This will help future maintainer of this code.

Thanks,

Vyom

On Tuesday 02 October 2018 06:39 AM, Prasadrao Koppula wrote:
Hi,

Could you please review this patch. This patch fixes the regression caused by 
LDAP fixes in JDK11, JDK8u181, JDK7u191

Webrev: http://cr.openjdk.java.net/~pkoppula/8211107/webrev.01/

Issue: https://bugs.openjdk.java.net/browse/JDK-8211107

Thanks,

Prasad.K


Reply via email to