On Wed, Nov 4, 2015 at 4:28 AM, Côme Chilliet <c...@opensides.be> wrote:

> Le mardi 3 novembre 2015, 18:59:42 Andrea Faulds a écrit :
> > I'd argue this is a regression, documented feature or not. Supporting
> > 'host:port' is not a bug, and this has broken existing code, which micro
> > (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 seems
> safer to require the scheme.
>
> Detecting the host:port syntax only to show a warning seems tedious. It’s
> like saying to the user «I get what you want me to do, but I won’t do it».
>
> And the point that it’s already been gone since a few version anyway is
> also a good point, people would have a code that works on some versions and
> not others, let’s 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 problem, so
> I’m for fixing the documentation on this one. Discussing the details on a
> separate email.
>
> MCMic
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
I would keep the old behavior for 5.6, even if that was unintended nobody
complained about it(so removing it isn't a bugfix per se), so I see no
reason to break userland code working before in a micro version.
for PHP-7.0 we can remove the old undocumented behavior but drop a mention
in NEWS/upgrading.

-- 
Ferenc Kovács
@Tyr43l - http://tyrael.hu

Reply via email to