Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:89064
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain opensides.be designates 195.154.20.141 as permitted sender)
To: internals@lists.php.net
Date: Wed, 04 Nov 2015 04:28:20 +0100
Message-ID: <2499781.S8xp2TXu1I@mcmic-probook>
Organization: OpenSides
User-Agent: KMail/4.14.1 (Linux/3.16.0-4-amd64; KDE/4.14.2; x86_64; ; )
In-Reply-To: <58.47.13519.54409365@pb1.pair.com>
References: <3187501.to7rK1FKUl@mcmic-probook> <58.47.13519.54409365@pb1.pair.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="utf-8"
Subject: Re: [PHP-DEV] Re: Small regression in PHP-LDAP
From: come@opensides.be (=?ISO-8859-1?Q?C=F4me?= Chilliet)

Le mardi 3 novembre 2015, 18:59:42 Andrea Faulds a =C3=A9crit :
> I'd argue this is a regression, documented feature or not. Supporting=
=20
> 'host:port' is not a bug, and this has broken existing code, which mi=
cro=20
> (x.y.Z) versions are not allowed to do. It should be fixed ASAP.

So I got two votes against and one for putting host:port back.

I feel bad about pissing off users that used host:port, but it does see=
ms safer to require the scheme.

Detecting the host:port syntax only to show a warning seems tedious. It=
=E2=80=99s like saying to the user =C2=ABI get what you want me to do, =
but I won=E2=80=99t do it=C2=BB.

And the point that it=E2=80=99s already been gone since a few version a=
nyway is also a good point, people would have a code that works on some=
 versions and not others, let=E2=80=99s save them the nightmare.

After reading the reactions on the bug tickets related to this problem,=
 it seems users are understanding the change and adapting without probl=
em, so I=E2=80=99m for fixing the documentation on this one. Discussing=
 the details on a separate email.

MCMic