Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:89051
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.170 as permitted sender)
MIME-Version: 1.0
Reply-To: bishop@php.net
Sender: bishop.bettini@gmail.com
In-Reply-To: <3187501.to7rK1FKUl@mcmic-probook>
References: <3187501.to7rK1FKUl@mcmic-probook>
Date: Tue, 3 Nov 2015 09:23:22 -0500
Message-ID: <CAEYWF=4too2Uv5fDsB_74VJttDK4J74VhM2Fox_j6nZ-yLnX7A@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=047d7bd91a0ad7019d0523a3a27f
Subject: Re: [PHP-DEV] Small regression in PHP-LDAP
From: bishop@php.net (Bishop Bettini)

--047d7bd91a0ad7019d0523a3a27f
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Tue, Nov 3, 2015 at 12:03 AM, C=C3=B4me Chilliet <come@opensides.be> wro=
te:

> So basically my question is, is that something that should be "fixed" to
> be consistent with previous PHP version, or should it stay that way as th=
is
> is what is documented and tested?
>

An equivalent question: Do you want to support this syntax going forward?

It's a common enough format that users likely will try it before reading
the documentation. But supporting it implies the code, the documentation,
and now a footnote that it wasn't supported between 5.6.11 and 5.6.x.

IMO, forbid it, adding a version note to the documentation: "5.6.11 --
Support for the undocumented $host format 'hostname:port' was removed."

bishop

--047d7bd91a0ad7019d0523a3a27f--