Fixing ldap_connect documentation
| From: | Côme Chilliet | 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.