Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:116492
Message-ID: <52b6fa2a-40cc-8716-2adb-2faeb569263e@iserv.eu>
Date: Wed, 24 Nov 2021 10:12:32 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.3.1
Content-Language: en-US
To: Jakub Zelenka <bukka@php.net>
Cc: PHP internals list <internals@lists.php.net>
References: <209120fa-1b09-a9e8-6873-89711f48830b@iserv.eu>
 <CAEKnhAFB=rUkmqYs56=Bvfa9h=Gkie_VmV9nsgVr0EZP3TTuzA@mail.gmail.com>
In-Reply-To: <CAEKnhAFB=rUkmqYs56=Bvfa9h=Gkie_VmV9nsgVr0EZP3TTuzA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Feedback on bugfix for streamed fcgi requests (#51191)
From: tim.siebels@iserv.eu (Tim Siebels)

Hi,

No worries. Thanks for the quick reply!

On 24.11.21 10:08, Jakub Zelenka wrote:
> On Wed, Nov 24, 2021 at 8:06 AM Tim Siebels <tim.siebels@iserv.eu> wrote:
> 
>> Hi everybody,
>>
>> I've submitted a pull-request to fix bug #51191. This is now waiting for
>> review for 2 months. Is there anything further I can do? What's the
>> appropriate duration to wait before pinging at this mailinglist?
>>
>> Link to PR: https://github.com/php/php-src/pull/7509
>>
>> I've just rebased to fix merge conflicts. CI is failing due to an error
>> on azure regarding purchased parallelism. I guess this will resolve
>> itself once this is fixed.
>>
>>
> Sorry I have been busy with that security issue in FPM which was a higher
> priority. The PR seems ready but need to think about the actual change a
> bit more. It's at least on my list now so should get resolved soon
> hopefully.
> 
> Thanks
> 
> Jakub
>