+1 On Dec 30, 2013, at 3:07 PM, Justin Erenkrantz <[email protected]> wrote:
> On Sun, Dec 29, 2013 at 3:48 PM, Jim Jagielski <[email protected]> wrote: >> Yeah, I'm leaning towards agreeing w/ you there. > > Both directives (HttpContentLengthHeadZero and HttpExpectStrict) > committed in r1554303. See below for how it affects the wire > protocol. > > Now, let's discuss the defaults. =) I'm +1 to the below patch. -- justin > > Index: modules/http/http_filters.c > =================================================================== > --- modules/http/http_filters.c (revision 1554304) > +++ modules/http/http_filters.c (working copy) > @@ -1254,7 +1254,7 @@ AP_CORE_DECLARE_NONSTD(apr_status_t) ap_http_heade > if (r->header_only > && (clheader = apr_table_get(r->headers_out, "Content-Length")) > && !strcmp(clheader, "0") > - && conf->http_cl_head_zero != AP_HTTP_CL_HEAD_ZERO_ENABLE) { > + && conf->http_cl_head_zero == AP_HTTP_CL_HEAD_ZERO_DISABLE) { > apr_table_unset(r->headers_out, "Content-Length"); > } >
