elecharny commented on PR #339: URL: https://github.com/apache/directory-server/pull/339#issuecomment-4339113655
So definitively the NoVerificationTrustManager should be removed, no question about that. It was created and used to simplify the TLS tests, without having to deal with the complexity of creating valid certificats. I still have to check what we have done in the past and what we should have done instead, but clearly there was room for improvement. Regarding the failing tests, I have to cchek now that MINA 2.2.6 has been release. I'm switching to cur a new LDAP API release in the coming days. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
