> On Jan 30, 2017, at 4:17 PM, Jim Jagielski <j...@jagunet.com> wrote:
> 
>> 
>> On Jan 30, 2017, at 4:04 PM, Jacob Champion <champio...@gmail.com> wrote:
>> 
>> On 01/28/2017 07:22 AM, Jim Jagielski wrote:
>>> In your scenario does "old mode" == "old" Apache or non Apache?
>> 
>> "Old" Apache.
>> 
>>> My idea was to send FCGI data such that PHP-FPM doesn't use *any*
>>> Apache-related fixups. In other words, httpd sends "what it should"
>>> and PHP-FPM "uses what it receives"
>> 
>> Right. My point is that without simultaneous modifications to PHP-FPM, I see 
>> no way to do this without breaking deployed use cases, because of how easily 
>> the fixup code is triggered. And therefore IMO we shouldn't push that change 
>> in 2.4.26, but wait (for a reasonable amount of time) for both sides to 
>> agree on the way forward.
>> 
>> I'll try to review Eric's patch later this week, since it would help ease 
>> the migration pain (and give other users a path forward even with broken 
>> FCGI backends).
>> 
> 
> Looking over fpm_main, Apache is detected iff PHP sees the proxy:balancer 
> and/or
> proxy:fcgi prefix. Looking at the logic paths related to 'apache_was_here',
> it looks like it just works around stuff that we *CAN* fix. For example, one
> code path is due to "mod_proxy_fcgi and ProxyPass, apache cannot set 
> PATH_INFO",
> but we can, and do now....

Plus, other than the "additional" way of detecting apache_was_here, that
file has been very stable with no real changes:

    https://github.com/php/php-src/commits/master/sapi/fpm/fpm/fpm_main.c

since

    
https://github.com/php/php-src/commit/e6d93a11ad343efdc42315f7f69ed82515c9f374#diff-624bdd47ab6847d777e15327976a9227

Reply via email to