RFR: 8301367: Add exception handler method to the BaseLdapServer

Daniel Fuchs dfuchs at openjdk.org
Thu Feb 2 09:56:21 UTC 2023


On Tue, 31 Jan 2023 16:19:34 GMT, Aleksei Efimov <aefimov at openjdk.org> wrote:

> The proposed change adds a new exception handler method to the `test/jdk/com/sun/jndi/ldap/lib/BaseLdapServer.java` LDAP test library class. It will allow LDAP tests to customize the handling of server-side exceptions.   
> The current `BaseLdapTestServer` implementation prints an exception and its stack trace to the standard error stream.
> 
> Existing tests in `test/jdk/com/sun/jndi/ldap` that use the modified library class are passing with the modified version.

Like Jaikiran I don't like the name of the new method much - but anything I could come up with was not much better. At least the new method gets both the socket and the exception as parameter, so that makes the name more reasonable. So approved as it stands!

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

Marked as reviewed by dfuchs (Reviewer).

PR: https://git.openjdk.org/jdk/pull/12347


More information about the core-libs-dev mailing list