Re: max-stale=0 forwarding

2008-03-21 Thread Henrik Nordstrom
On Fri, 2008-03-21 at 10:08 -0400, Matt Benjamin wrote: Today, just request-side handling of Cache-Control: max-stale=0 (HttpHdrCc.c, 2.6-STABLE7+). If request's Cache-Control: max-stale has any positive value, Squid passes it through. Shouldn't the allowed range include 0 (delta-seconds

Re: max-stale=0 forwarding

2008-03-21 Thread Matt Benjamin
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi Henrik, 1. I'll file a bug. I have the patch for it too, of course 2. The client in the case we've run into seems to be a network device, a proxy or traffic-management server--google finds examples of other folks doing it 3. The current code

Re: max-stale=0 forwarding

2008-03-21 Thread Henrik Nordstrom
On Fri, 2008-03-21 at 18:16 -0400, Matt Benjamin wrote: 3. The current code is problematic because the spec has max-stale without a value equivalent to max-stale: (Infinity) Oh, I thought it didn't forward max-stale=0 at all. Forwarding it as max-stale without a delta is defenitely a bug. The

Re: max-stale=0 forwarding

2008-03-21 Thread Henrik Nordstrom
On Fri, 2008-03-21 at 23:33 +0100, Henrik Nordstrom wrote: On Fri, 2008-03-21 at 18:16 -0400, Matt Benjamin wrote: 3. The current code is problematic because the spec has max-stale without a value equivalent to max-stale: (Infinity) Oh, I thought it didn't forward max-stale=0 at all.