8177806: ldapContext.unbind not handling ldap entry with esc char
rob.mckenna at oracle.com
Thu Jul 6 15:57:44 UTC 2017
Thanks Vyom, updated test at:
The sleep proved necessary in a similar earlier test so I'll leave it in
there. (iirc the request failed as it was sent before the server started
On 06/07/17 01:51, Vyom Tewari wrote:
> Hi Rob,
> 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
> 3-> Do you need Thread.sleep(500) ?. I think it is not required.
> 4-> i will suggest you to use below tag order.
> * @test
> * @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:
> > -Rob
More information about the core-libs-dev