Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:81015
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.46 as permitted sender)
MIME-Version: 1.0
Sender: yohgaki@gmail.com
In-Reply-To: <54C1261D.7080009@php.net>
References: <54C1261D.7080009@php.net>
Date: Fri, 23 Jan 2015 12:42:00 +0900
Message-ID: <CAGa2bXbCkFT+vkNBe-RZgM9TR72OHZHQ67UG3DMFC9RiyEP1+w@mail.gmail.com>
To: Michael Wallner <mike@php.net>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c22ccc9b7529050d4994a0
Subject: Re: [PHP-DEV] [RFC] pecl_http
From: yohgaki@ohgaki.net (Yasuo Ohgaki)

--001a11c22ccc9b7529050d4994a0
Content-Type: text/plain; charset=UTF-8

Hi Mike,

On Fri, Jan 23, 2015 at 1:32 AM, Michael Wallner <mike@php.net> wrote:

> Now, that I'm mostly done with porting pecl/http [1] and dependencies
> (propro [2] and raphf [3]) to ZE3 I'd like to restart discussion on the
> topic, whether it is feasible to add pecl_http as a bundled extension to
> the core.
>
> https://wiki.php.net/rfc/pecl_http
>

Could you include http_build_query() modification in the RFC?
http_build_query() escapes ' ' as '+' currently. It should be '%20'.
I was about to proposing this change, but it was http_* function and
the change does not break scripts.

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net

--001a11c22ccc9b7529050d4994a0--