httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
Date Tue, 17 Aug 2010 15:22:08 GMT

On Aug 17, 2010, at 10:55 AM, Rainer Jung wrote:

> On 17.08.2010 16:43, jim@apache.org wrote:
>> Author: jim
>> Date: Tue Aug 17 14:43:45 2010
>> New Revision: 986333
>> 
>> URL: http://svn.apache.org/viewvc?rev=986333&view=rev
>> Log:
>> Further checks for non-body requests...
>> 
>> Modified:
>>     httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
>> 
>> Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
>> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_http.c?rev=986333&r1=986332&r2=986333&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/modules/proxy/mod_proxy_http.c (original)
>> +++ httpd/httpd/trunk/modules/proxy/mod_proxy_http.c Tue Aug 17 14:43:45 2010
>> @@ -704,8 +704,15 @@ int ap_proxy_http_request(apr_pool_t *p,
>>       * Send the HTTP/1.1 request to the remote server
>>       */
>> 
>> +    /*
>> +     * To be compliant, we only use 100-Continue for requests with no bodies.
>> +     * We also make sure we won't be talking HTTP/1.0 as well.
>> +     */
>>      do_100_continue = (worker->ping_timeout_set
>>                         &&  !r->header_only
>> +&&  !r->kept_body
>> +&&  !(apr_table_get(r->headers_in, "Content-Length"))
>> +&&  !(apr_table_get(r->headers_in, "Transfer-Encoding"))
> 
> Did you mean:
> 
> && (!apr_table_get(r->headers_in, "Content-Length") ||
>    !apr_table_get(r->headers_in, "Transfer-Encoding")
> 

I think you meant:

  && (!(apr_table_get(r->headers_in, "Content-Length") || apr_table_get(r->headers_in,
"Transfer-Encoding")))

Which xforms to

  && !a
  && !b


Mime
View raw message