Re: [MERGE] Squid Patch (revision 10487)

On 2014-05-13 11:53, Amos Jeffries wrote:
> On 13/05/2014 10:52 p.m., Amos Jeffries wrote:
>> On 13/05/2014 4:56 a.m., Michael Pye wrote:
>>> On 2014-05-12 16:52, Amos Jeffries wrote:
>>>> Here is the guts of the original patch as should be relevant to
>>>> Squid-3.5 (or 3.4).
>>>> http://master.squid-cache.org/~amosjeffries/patches/Pye_s35.patch
>>>
>>> Many thanks Amos, I've just tried this and it does indeed query
>>> sibling
>>> for stale content as we want.
>>>
>>> I think there is some logic that will need updating however in
>>> HttpHeader.cc possibly, as when an object becomes stale and the
>>> sibling
>>> does not have a fresh copy, the origin is queried but the "Age:"
>>> and
>>> "X-Cache:HIT from" headers are still on the new request, even
>>> though it
>>> was a fresh check from the origin. I think these headers need to be
>>> deleted/updated.
>>
>> Is this a fully new set of headers+object from the server?
>> or a revalidation update/refresh from the server?

It's a revalidation, so it sends an "if-modified-since" to the origin
which replies with a "304 not modified". At this point it seems the Age:
needs recalculating.

Thanks
Michael
Received on Tue May 13 2014 - 15:46:11 MDT