Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:56291
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.161.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4EBD8967.8040606@lerdorf.com>
References: <1320740449-sup-5688@ezyang>
	<4EB8EAF1.3040104@lerdorf.com>
	<1320981061-sup-2803@ezyang>
	<4EBD79A5.2020604@lerdorf.com>
	<1321040435-sup-2225@ezyang>
	<4EBD8967.8040606@lerdorf.com>
Date: Fri, 11 Nov 2011 22:42:11 +0100
Message-ID: <CAEZPtU4Bahm4Jak2o+CqsAx=eM61y6-2x4Ta=hSDxBA_DR0e7w@mail.gmail.com>
To: Rasmus Lerdorf <rasmus@lerdorf.com>
Cc: "Edward Z. Yang" <ezyang@mit.edu>, internals <internals@lists.php.net>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] Load extensions for .user.ini files
From: pierre.php@gmail.com (Pierre Joye)

hi,

On Fri, Nov 11, 2011 at 9:45 PM, Rasmus Lerdorf <rasmus@lerdorf.com> wrote:

> The other issue with your patch is that you are enabling this for the
> fpm sapi. I thought we agreed that this should only be in the sapis that
> supported dl() and those are cgi, cli and embed. And .user.ini doesn't
> apply to embed, so it would just be cli and cgi because these are the
> non-persistent cases where there is no danger of screwing up the next
> request with a bad extension in a .user.ini file.

Right, that's what I worry about this patch. I would not introduce
this patch for anything but where dl is available.

Cheers,
-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org