<html>

  <head>

    <meta http-equiv="content-type" content="text/html; charset=utf-8">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <meta http-equiv="content-type" content="text/html; charset=utf-8">

    <pre style="color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: 1; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; white-space: pre-wrap;">Hello,

This patch forces Squid to always revalidate Cache-Control:no-cache

responses.

Squid MUST NOT use a CC:no-cache response for satisfying subsequent

requests without successful validation with the origin server. Squid

violated this MUST because Squid mapped both "no-cache" and

"must-revalidate"/etc. directives to the same ENTRY_REVALIDATE flag

which was interpreted as "revalidate when the cached response becomes

stale" rather than "revalidate on every request".

This patch splits ENTRY_REVALIDATE into:

- ENTRY_REVALIDATE_ALWAYS, for entries with CC:no-cache and

- ENTRY_REVALIDATE_STALE, for entries with other related CC directives

  like CC:must-revalidate and CC:proxy-revalidate.

Reusing ENTRY_CACHABLE_RESERVED_FOR_FUTURE_USE value for the more

forgiving ENTRY_REVALIDATE_STALE (instead of the more aggressive

ENTRY_REVALIDATE_ALWAYS) fixes the bug for the already cached entries -

they will be revalidated and stored with the correct flag on the next

request.

Regards,

Eduard.

</pre>

  </body>

</html>