Re: [PHP-DEV] Fixing ldap_connect documentation

2015-11-04 Thread Côme Chilliet
Does this seems correct? https://github.com/php/php-src/pull/1618 -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] Fixing ldap_connect documentation

2015-11-04 Thread Côme Chilliet
Le mercredi 4 novembre 2015, 21:10:09 Bishop Bettini a écrit : > > - 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. > > > > I'd prefer this raise a notice, like "Second argument ignored when using > U

Re: [PHP-DEV] Fixing ldap_connect documentation

2015-11-04 Thread Bishop Bettini
On Tue, Nov 3, 2015 at 11:00 PM, Côme Chilliet wrote: > 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 > Agreed. > - Second parameter i