Fixing ldap_connect documentation

From: Date: Wed, 04 Nov 2015 04:00:38 +0000
Subject: Fixing ldap_connect documentation
References: 1  Groups: php.internals 
Request: Send a blank email to internals+get-89065@lists.php.net to get a copy of this message
Hello, About ldap_connect documentation, we got an interesting proposition there: https://github.com/owncloud/core/issues/20020#issuecomment-153387886 The idea is to documment the LDAP URI version as a default and only list the host, port syntax as a backward compatibilty thing. Which it is IMO. The fact that when using (host, port) SSL is not supported goes strongly in that direction. http://php.net/manual/en/ldap.requirements.php already says OpenLDAP>=2.4 is needed so «If you are using OpenLDAP 2.x.x» can be removed. What should be clear in the documentation : - The LDAP URI scheme is the recommended way to call ldap_connect - The ldap:// or ldaps:// is required - People can use host, port if they do not need SSL - Second parameter is not used if the first one is an LDAP scheme, so calling ldap_connect('ldap://localhost', 456) will use default port 389. Does that seem ok? MCMic PS: just saw that Andreas Heigl already edited the doc, to some of it is already fixed. It could still be improved to recommend LDAP URIS more clearly.

« previous php.internals (#89065) next »