Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:89070
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.177 as permitted sender)
MIME-Version: 1.0
Reply-To: bishop@php.net
Sender: bishop.bettini@gmail.com
In-Reply-To: <7863331.7prpVflOvN@mcmic-probook>
References: <3187501.to7rK1FKUl@mcmic-probook>
	<7863331.7prpVflOvN@mcmic-probook>
Date: Wed, 4 Nov 2015 21:10:09 -0500
Message-ID: <CAEYWF=7aEbhDA7xSqQWuGO61myPjfr_t4VUsxwFmkG2LkpnRCA@mail.gmail.com>
To: =?UTF-8?Q?C=C3=B4me_Chilliet?= <come@opensides.be>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7bd91a0a5c62ed0523c1a00c
Subject: Re: [PHP-DEV] Fixing ldap_connect documentation
From: bishop@php.net (Bishop Bettini)

--047d7bd91a0a5c62ed0523c1a00c
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Tue, Nov 3, 2015 at 11:00 PM, C=C3=B4me Chilliet <come@opensides.be> wro=
te:

> What should be clear in the documentation=E2=80=AF:
> - 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 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
URI syntax. Port will default to 389. Did you mean 'ldap://localhost:456'
instead?"

bishop

--047d7bd91a0a5c62ed0523c1a00c--