[8u] RFR 8062947: Fix exception message to correctly represent LDAP connection failure

Andrew Hughes gnu.andrew at redhat.com
Mon Sep 7 05:50:11 UTC 2020


On 11:45 Wed 02 Sep     , Zhengyu Gu wrote:

snip...

> 
> The original bug: https://bugs.openjdk.java.net/browse/JDK-8062947
> The original patch: https://hg.openjdk.java.net/jdk/jdk/rev/868fe697bad4
> 
> 
> 8u webrev: http://cr.openjdk.java.net/~zgu/JDK-8062947-8u/webrev.00/
> 
> Test:
>   Included NamingExceptionMessageTest.java test on Linux x86_64
> 
> Thanks,
> 
> -Zhengyu
> 

The posted patch -
https://cr.openjdk.java.net/~zgu/JDK-8062947-8u/webrev.00/ - seems to
contain multiple changes unrelated to this bug:

$ diffstat -p0 8062947.11u
b/src/java.naming/share/classes/com/sun/jndi/ldap/Connection.java | 65 +--
b/src/java.naming/share/classes/com/sun/jndi/ldap/LdapRequest.java | 27 +
b/test/jdk/com/sun/jndi/ldap/NamingExceptionMessageTest.java | 164 ++++++++++
3 files changed, 205 insertions(+), 51 deletions(-)

$ diffstat -p0 8062947.8u
b/src/share/classes/com/sun/jndi/ldap/Connection.java | 65
b/src/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java | 4
b/src/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java | 25
b/src/share/classes/com/sun/jndi/ldap/LdapRequest.java | 27
b/test/com/sun/jndi/ldap/LdapTimeoutTest.java | 778 +++++-----
b/test/com/sun/jndi/ldap/NamingExceptionMessageTest.java | 165 ++
b/test/com/sun/jndi/ldap/lib/BaseLdapServer.java | 27
7 files changed, 668 insertions(+), 423 deletions(-)
	
Please post a webrev just containing this fix.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222


More information about the jdk8u-dev mailing list