Re: Pre-release test tarballs of httpd 1.3.40, 2.0.62 and 2.2.7 available

[Ruediger Pluem]

On 01/05/2008 11:02 AM, Nick Kew wrote:
> On Sat, 05 Jan 2008 10:37:53 +0100
> Ruediger Pluem <[EMAIL PROTECTED]> wrote:
> 
>>
>> On 01/05/2008 02:02 AM, Nick Kew wrote:
>>> On Sat, 5 Jan 2008 00:07:42 +0000
>>> Nick Kew <[EMAIL PROTECTED]> wrote:
>>>
>>>> Regression: fails to proxy massively chunked responses.
>>> OK, it works correctly if I revert r602679 (no other changes).
>>> Investigating further.
>> It would be helpful if you could provide us with more details what
>> exactly fails and if possible with a test case.
> 
> http://people.apache.org/~niq/TestLogWebViewer.html
> 
> (note: s/100Byte/1Byte in the test case description, as is clear
>>from the trace).
> 


Can you please try if the following patch against the vanilla tar ball
fixes the problem?

Index: modules/http/http_filters.c
===================================================================
--- modules/http/http_filters.c (Revision 609114)
+++ modules/http/http_filters.c (Arbeitskopie)
@@ -321,7 +321,8 @@
                                         block, 0);
                     apr_brigade_cleanup(bb);
                     if (block == APR_NONBLOCK_READ &&
-                        (APR_STATUS_IS_EAGAIN(rv))) {
+                        ( (rv == APR_SUCCESS && APR_BRIGADE_EMPTY(bb)) ||
+                          (APR_STATUS_IS_EAGAIN(rv)) )) {
                         return APR_EAGAIN;
                     }
                 } else {


Regards

Rüdiger