8177806: ldapContext.unbind not handling ldap entry with esc char
vyom.tewari at oracle.com
Thu Jul 6 08:21:47 UTC 2017
Code change looks OK, below are few comments on test case.
1-> Remove System.exit(1) and throw Exception from main if test fails.
2-> Put srv.close() in finally block, in case of exception close will
not be called.
3-> Do you need Thread.sleep(500) ?. I think it is not required.
4-> i will suggest you to use below tag order.
* @bug 8177806
* @summary Test NameImpl escape character processing
* @modules java.naming/javax.naming:open
* @run main/othervm LdapEscapeTest
On Thursday 06 July 2017 04:14 AM, Rob McKenna wrote:
> Hi folks,
> Looking for a review for the following change.
> NameImpl.java is stripping escape characters unnecessarily:
More information about the core-libs-dev