[9] request for review 8059009: LDAPCertStore fails to retrieve CRL after LDAP server closes idle connection

Tue Jan 13 11:08:33 UTC 2015

Your fix looks good.
Thanks.

On 1 Dec 2014, at 18:10, Artem Smotrakov <artem.smotrakov at oracle.com> wrote:

> Please review this fix for 9.
> 
> It corrects the following issues:
> - InitialLdapContext and LDAPCertStore does not recover connection if it was closed, for example, by server, or due to some network issue
> - A cache of CertStore instances in LDAPCertStore does not work if LDAPCertStoreParameters is used
> 
> Changes:
> - updated LdapCtx to make re-connect possible
> - updated LDAPCertStore to use InitialLdapContext that supports re-connect
> - updated LDAPCertStore to use only SunLDAPCertStoreParameters objects for cache of CertStore instances
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8059009
> Webrev: http://cr.openjdk.java.net/~asmotrak/8059009/webrev.01/
> 
> Artem