Your fix looks good. Thanks.
On 1 Dec 2014, at 18:10, Artem Smotrakov <artem.smotra...@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
