In a message dated 01-09-10 12:28:55 EDT, Kevin Kiley wrote... > The following is NOT flamebait. I swear. > It is just an observation that is missing from the discussion. > > I am just pointing out that no one has done a really good > code review of mod_gz even if the 'consensus' is to drop > it into the core. I've already pointed out one or two problems > ( minor ) but a new one I found is that it seems to duplicate > the addition of the GZIP header at all times ( gets added twice?) > before the EOS bucket is sent down the pike. Client-side results > could be unpredictable. Depends on the browser. It is also completely > ignoring the FLUSH bucket when it arrives and has no way to add > 'Content-length:' if/when it is necessary. Erratta... I meant to say that the GZIP 'footer' containing OCL ( Original Content Length ) and CRC seems to be duplicated on each call... the HEADER only seems to go in once as it should. Sorry about that. Later... Kevin
- Re: cvs commit: httpd-2.0 STATUS Rodent of Unusual Size
- Re: cvs commit: httpd-2.0 STATUS Greg Stein
- Re: cvs commit: httpd-2.0 STATUS Rodent of Unusual Size
- Re: cvs commit: httpd-2.0 STATUS Greg Stein
- RE: cvs commit: httpd-2.0 STATUS Joshua Slive
- Re: cvs commit: httpd-2.0 STATUS Greg Stein
- Re: cvs commit: httpd-2.0 STATUS TOKILEY
- why mod_gz "now" ? (was: Re: cvs com... Greg Stein
- Re: cvs commit: httpd-2.0 STATUS William A. Rowe, Jr.
- RE: cvs commit: httpd-2.0 STATUS Joshua Slive
- Re: cvs commit: httpd-2.0 STATUS TOKILEY
- Re: cvs commit: httpd-2.0 STATUS Greg Stein
- Re: cvs commit: httpd-2.0 STATUS Ryan Bloom
- Re: cvs commit: httpd-2.0 STATUS Cliff Woolley
- Re: cvs commit: httpd-2.0 STATUS Justin Erenkrantz
- Re: cvs commit: httpd-2.0 STATUS Jeff Trawick
- Re: cvs commit: httpd-2.0 STATUS Justin Erenkrantz
- Re: cvs commit: httpd-2.0 STATUS Rodent of Unusual Size
- Re: cvs commit: httpd-2.0 STATUS Rodent of Unusual Size
- coding style cleanup Re: cvs commit: httpd-2.0... Brian Pane
- Re: coding style cleanup Re: cvs commit: h... Justin Erenkrantz