Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:76699
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain schlueters.de from 217.114.215.10 cause and error)
To: Michael Wallner <mike@php.net>
Cc: PHP Internals <internals@lists.php.net>
In-Reply-To: <53F3018E.6090201@php.net>
References: <53F3018E.6090201@php.net>
Content-Type: text/plain; charset="UTF-8"
Date: Tue, 19 Aug 2014 13:53:14 +0200
Message-ID: <1408449194.2617.375.camel@guybrush>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] pecl_http
From: johannes@schlueters.de (Johannes =?ISO-8859-1?Q?Schl=FCter?=)

On Tue, 2014-08-19 at 09:49 +0200, Michael Wallner wrote:
> Hi PHP internals!
> 
> I've created an RFC for discussion:
> 
> 	Whether it is feasible to add pecl_http v2 to the core.
> 
> https://wiki.php.net/rfc/pecl_http

In my opinion when moving this to core the http(s):// stream should use
the same HTTP implementation so one has consistent behavior.

I also see that HTTP offers "Content type guessing [...] by building
this extension with libmagic support" I think that should based on
fileinfo so both report the same type for a file. Again consistency.

I also think the HTTP module should be forced on (like SPL or pcre) or
at least be on by default (unless --disable-[all|http] is used)

johannes