Revert "Added back support for undocummented host:port syntax"

This reverts commit 15876e85e59658bfbda3bf0b0c34d56ca503fd2f.

diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
index 64885c7cb73a0acefe407b99401ee1b71e97193f..ab663b156be320c3bba8a7267187105adc47361a 100644 (file)
--- a/ext/ldap/ldap.c
+++ b/ext/ldap/ldap.c
@@ -369,12 +369,7 @@ PHP_FUNCTION(ldap_connect)
                        }
 
                        url = emalloc(urllen);
-                       if (host && (strchr(host, ':') != NULL)) {
-                               /* Legacy support for host:port */
-                               snprintf( url, urllen, "ldap://%s", host );
-                       } else {
-                               snprintf( url, urllen, "ldap://%s:%ld", host ? host : "", port );
-                       }
+                       snprintf( url, urllen, "ldap://%s:%ld", host ? host : "", port );
                }
 
 #ifdef LDAP_API_FEATURE_X_OPENLDAP

diff --git a/ext/ldap/tests/ldap_connect_variation.phpt b/ext/ldap/tests/ldap_connect_variation.phpt
index eaee261542514ef6102113d2d5003df5e92e7e61..09b07e778630f0fd5facffd9fb64fdf040f9324a 100644 (file)
--- a/ext/ldap/tests/ldap_connect_variation.phpt
+++ b/ext/ldap/tests/ldap_connect_variation.phpt
@@ -28,10 +28,6 @@ var_dump($link);
 // bad hostname (connect should work, not bind)
 $link = ldap_connect("nonexistent" . $host);
 var_dump($link);
-
-// Legacy host:port syntax
-$link = ldap_connect("$host:$port");
-var_dump($link);
 ?>
 ===DONE===
 --EXPECTF--
@@ -40,5 +36,4 @@ resource(%d) of type (ldap link)
 resource(%d) of type (ldap link)
 resource(%d) of type (ldap link)
 resource(%d) of type (ldap link)
-resource(%d) of type (ldap link)
 ===DONE===