On Wed, 17 Nov 2021 20:04:50 GMT, Martin Balao <mbalao@openjdk.org> wrote:
Hi Martin,
The change looks reasonable to me. I would suggest having a CSR logged for this change due to the following [behavioral incompatibility](https://wiki.openjdk.java.net/display/csr/Kinds+of+Compatibility): Before the change - all available endpoints/URLs are tried to create an LDAP context. With the proposed change - incorrect credentials will prevent other endpoints to be exercised to create an LDAP context.
Having a CSR will also help to document difference in handling `AuthenticationException` and `NamingException` during construction of an LDAP context from the list of endpoints acquired from a LDAP DNS provider.
Hi @AlekseiEfimov
Can you please review the CSR [1]?
Thanks, Martin.-
@martinuy This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply add a new comment to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!
Please do not close, waiting for CSR approval. ------------- PR: https://git.openjdk.java.net/jdk/pull/6043