Integrated: 8273402: Use derived NamingExceptions in com.sun.jndi.ldap.Connection#readReply

Aleksei Efimov aefimov at openjdk.java.net
Fri Sep 10 14:18:53 UTC 2021


On Thu, 9 Sep 2021 22:02:55 GMT, Aleksei Efimov <aefimov at openjdk.org> wrote:

> Hi,
> The following fix changes type of exception thrown when an LDAP operation fails for reasons like read timeout or connection closure/cancellation: instead of throwing a general `NamingException`, the internal LDAP connection class will throw a [`CommunicationException`](https://github.com/openjdk/jdk/blob/master/src/java.naming/share/classes/javax/naming/CommunicationException.java#L29) that better matches the reasons described.
> 
> Testing shows no problem with the proposed fix.

This pull request has now been integrated.

Changeset: c464f090
Author:    Aleksei Efimov <aefimov at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c464f09056c239f701b400a5c59c54646f840391
Stats:     41 lines in 3 files changed: 13 ins; 3 del; 25 mod

8273402: Use derived NamingExceptions in com.sun.jndi.ldap.Connection#readReply

Reviewed-by: dfuchs

-------------

PR: https://git.openjdk.java.net/jdk/pull/5456


More information about the core-libs-dev mailing list