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.

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to