httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mladen Turk <mt...@apache.org>
Subject Re: svn commit: r696614 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_ajp.c
Date Fri, 19 Sep 2008 09:18:00 GMT
Ruediger Pluem wrote:
> 
>> +    if (r->clength > 0) {
>> +        return r->clength;
>> +    }
>> +    else if (r->main == NULL || r->main == r) {
> 
> There are subrequests that have themselves as parent?
>

It's code from mod_jk. Can be overengineering.

> 
> But this does mean that the container receives a request, but we doesn't 
> read the answer.
> Is the container capable of handling this situation?

Yes. Zero length body means disconnected client (per AJP protocol)

> Or does it simply error out without sending anything and keeping the 
> connection open on
> requests with C-L set and a null length data packet sent?
> 

Without the patch the container side will presume the next packet
(can be new request or ping) is first body packet.

Regards
-- 
^(TM)

Mime
View raw message