Bug #426: "Vary: *" ignored

"Vary: *" headers was ingored, possibly returning unacceptable cache hits
if such header is present.

diff --git a/src/http.cc b/src/http.cc
index 33ca9f6daa9688a5d53b0c82edd93dfcd53c782a..73db0159b56bb5fdae130b26a1948daab1e81f7e 100644 (file)
--- a/src/http.cc
+++ b/src/http.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: http.cc,v 1.429 2003/10/22 21:40:40 robertc Exp $
+ * $Id: http.cc,v 1.430 2004/04/03 14:17:36 hno Exp $
  *
  * DEBUG: section 11    Hypertext Transfer Protocol (HTTP)
  * AUTHOR: Harvest Derived
@@ -532,6 +532,14 @@ httpMakeVaryMark(HttpRequest * request, HttpReply const * reply)
         char *name = (char *)xmalloc(ilen + 1);
         xstrncpy(name, item, ilen + 1);
         Tolower(name);
+
+        if (strcmp(name, "*") == 0) {
+            /* Can not handle "Vary: *" withtout ETag support */
+            safe_free(name);
+            vstr.clean();
+            break;
+        }
+
         strListAdd(&vstr, name, ',');
         hdr = httpHeaderGetByName(&request->header, name);
         safe_free(name);