From: wessels <>
Date: Sat, 18 Jul 1998 13:28:03 +0000 (+0000)
Subject: cachability for responses from authenticated requests should depend
X-Git-Tag: SQUID_3_0_PRE1~3053
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=a6dfe2d9570ee4c52103fcd4f6bab9e0398aea1b;p=thirdparty%2Fsquid.git

cachability for responses from authenticated requests should depend
upon the presence of Cache-Control: Public, and not Cache-Control:
Proxy-Revalidate.
---

diff --git a/src/http.cc b/src/http.cc
index 88fb2a7f1b..46d949ec2e 100644
--- a/src/http.cc
+++ b/src/http.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: http.cc,v 1.291 1998/07/16 22:22:49 wessels Exp $
+ * $Id: http.cc,v 1.292 1998/07/18 07:28:03 wessels Exp $
  *
  * DEBUG: section 11    Hypertext Transfer Protocol (HTTP)
  * AUTHOR: Harvest Derived
@@ -216,9 +216,15 @@ httpCachableReply(HttpStateData * httpState)
 	return 0;
     if (EBIT_TEST(cc_mask, CC_NO_STORE))
 	return 0;
-    if (EBIT_TEST(httpState->request->flags, REQ_AUTH))
-	if (!EBIT_TEST(cc_mask, CC_PROXY_REVALIDATE))
+    if (EBIT_TEST(httpState->request->flags, REQ_AUTH)) {
+	/*
+	 * Responses to requests with authorization may be cached
+	 * only if a Cache-Control: pubic reply header is present.
+	 * RFC 2068, sec 14.9.4
+	 */
+	if (!EBIT_TEST(cc_mask, CC_PUBLIC))
 	    return 0;
+    }
     /*
      * We don't properly deal with Vary features yet, so we can't
      * cache these