> From: Greg Marr [mailto:[EMAIL PROTECTED]] > > At 10:10 PM 04/11/2002, Ryan Bloom wrote: > > > From: Justin Erenkrantz [mailto:[EMAIL PROTECTED]] > > > On Thu, Apr 11, 2002 at 07:02:48PM -0700, Ryan Bloom wrote: > > > > Dollars to Donuts, the problem is that the C-L filter isn't > > removing the > > > > C-L header from the request. I won't have time to look at this > > for a > > > > few weeks though. > > > > > > The request? Don't you mean the file? > > > > > > Taking a quick look at default_handler: > > > > > > server/core.c line 3208: > > > ap_set_content_length(r, r->finfo.size); > > > > > > Isn't that call bogus? Why just not let the C-L filter handle it? > > > >No, it isn't bogus. If the file isn't changed, then the call is > >correct. It is only bogus if the content is changed, as it is with > >SSI requests. If the C-L filter can't verify the C-L, it should be > >removing it from the request. If it doesn't, that is the bug. > > Isn't the filter that changes the content responsible for removing > the C-L? Otherwise, the C-L filter would either remove the C-L every > time, or have to buffer the entire thing.
There are people who believe that the filter that modifies the content should be responsible for removing the C-L (I happen to be one of them), but that is not the current design. Ryan
