Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:69783
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.45 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <52603390.7020609@ajf.me>
References: <52603390.7020609@ajf.me>
Date: Tue, 22 Oct 2013 21:51:03 +0100
Message-ID: <CAPwsocH_4LUFXriCqTcfvOb4r5vdqqKEnE3SR=qzR5yD7Zt0nw@mail.gmail.com>
To: Andrea Faulds <ajf@ajf.me>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] [PATCH] Add apache_request_headers() and
 apache_response_headers() to the CLI server
From: leight@gmail.com (Leigh)

On 17 October 2013 19:59, Andrea Faulds <ajf@ajf.me> wrote:
> Hello all,
>
> I implemented FR #65917 (getallheaders() is not supported by the built-in
> web server). FR #65917 is actually my own feature request; I created it
> because I had tried to use it with the CLI server to test an application of
> mine, and it wasn't available. So I went and implemented it, and since
> apache_response_headers() complements it and was trivial to implement as
> well, I have also implemented that, by copying the FastCGI code.

Sorry for a late response. I wanted to add my personal feeling is that
I do not like apache_* functions (or function aliases) existing
outside of the Apache SAPI.

getallheaders() yes
apache_* no

I decided to click the pull request link before hitting send. Looks
like this has already been brought up. Going to hit send anyway :)