You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2015/03/31 00:52:53 UTC

svn commit: r1670208 - /directory/shared/trunk/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java

Author: elecharny
Date: Mon Mar 30 22:52:52 2015
New Revision: 1670208

URL: http://svn.apache.org/r1670208
Log:
Fix for DIRAPI-230 : we just don't wait for obvious wrong connections that will never be better if we wait

Modified:
    directory/shared/trunk/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java

Modified: directory/shared/trunk/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java?rev=1670208&r1=1670207&r2=1670208&view=diff
==============================================================================
--- directory/shared/trunk/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java (original)
+++ directory/shared/trunk/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java Mon Mar 30 22:52:52 2015
@@ -582,7 +582,10 @@ public class LdapNetworkConnection exten
 
                     if ( !isConnected )
                     {
-                        if ( connectionFuture.getException() instanceof ConnectException )
+                        Throwable connectionException = connectionFuture.getException();
+
+                        if ( ( connectionException instanceof ConnectException )
+                            || ( connectionException instanceof UnresolvedAddressException ) )
                         {
                             // No need to wait
                             // We know that there was a permanent error such as "connection refused".