Re: [squid-users] Re: not caching ecap munged body

2011-02-03 Thread Amos Jeffries
Sounds good. Looks good. If you can post that to squid-dev today it can get into the merge process before next release. We need: * attached file suitable for applying with "patch -p0" * subject line having [PATCH] then a short Changelog label * a brief description of the problem for commit m

Re: [squid-users] Re: not caching ecap munged body

2011-02-03 Thread Jonathan Wolfe
Here's the patch - one extra line necessary to make the comparison actually work. Shall I file a squid bug for ecap modules removing Content-Length with this patch attached? After patching, squid properly caches an adapted response with a different content length from the virgin response. Wit

Re: [squid-users] Re: not caching ecap munged body

2011-02-02 Thread Amos Jeffries
On Wed, 2 Feb 2011 20:09:05 -0800, Jonathan Wolfe wrote: > Oops, here's the diff from Alex which I forgot to include below: > > === modified file 'src/adaptation/ecap/MessageRep.cc' > --- src/adaptation/ecap/MessageRep.cc 2010-05-26 03:06:02 + > +++ src/adaptation/ecap/MessageRep.cc 2011-01-26

[squid-users] Re: not caching ecap munged body

2011-02-02 Thread Jonathan Wolfe
Oops, here's the diff from Alex which I forgot to include below: === modified file 'src/adaptation/ecap/MessageRep.cc' --- src/adaptation/ecap/MessageRep.cc 2010-05-26 03:06:02 + +++ src/adaptation/ecap/MessageRep.cc 2011-01-26 21:43:36 + @@ -44,24 +44,30 @@ void Adaptation::Ecap::Header