On Wed, Oct 21, 2015 at 4:22 PM, Plüm, Rüdiger, Vodafone Group <ruediger.pl...@vodafone.com> wrote: > > >> -----Ursprüngliche Nachricht----- >> Von: Yann Ylavic [mailto:ylavic....@gmail.com] >> Gesendet: Mittwoch, 21. Oktober 2015 16:07 >> An: dev@httpd.apache.org >> Betreff: Re: [users@httpd] Chunked transfer delay with httpd 2.4 on >> Windows. >> >> On Wed, Oct 21, 2015 at 2:29 PM, Ruediger Pluem <rpl...@apache.org> >> wrote: >> > >> > This looks like there is a stray \n in the input queue that causes >> httpd to think that there is a pipelined request. >> >> I think we should tolerate blank lines in check_pipeline(), like >> read_request_line() does (this is also a RFC compliance). >> >> How about the following patch? > > In general this looks good, but why not moving the max_blank_lines logic > into check_pipeline using c->server->limit_req_fields, so that we do not need > to change > its prototype?
Hmm, check_pipeline() is static, why bother? Also c->base_server may be different than r->server (after the first request), and we probably want to use the value from the last request's vhost (a bit like we did already for keep_alive_timeout). Regards, Yann.