Changeset 1506 for trunk/varnish-cache/bin/varnishd/cache_hash.c

Timestamp:

06/10/07 10:49:43 (3 years ago)

Author:

phk

Message:

Take a shot at light-weight "Vary:" processing.

When we cache an object with a "Vary:" header, we generate
a "vary matching string" which can be used to efficiently
check for compliance when doing a cache lookup.

Only very lightly tested (ie: cnn.com).

For a full description of the reasoning, please see

 http://varnish.projects.linpro.no/wiki/ArchitectureVary

Files:

• trunk/varnish-cache/bin/varnishd/cache_hash.c (modified) (3 diffs)

trunk/varnish-cache/bin/varnishd/cache_hash.c

@@ -149 +149 @@
                 }
         were_back:
-                /* XXX: check Vary: */
                 if (!o->cacheable) {
                         /* ignore */
@@ -160 +159 @@
                         VSL(SLT_ExpBan, 0, "%u was banned", o->xid);
                         EXP_TTLchange(o);
-                } else
+                } else if (VRY_Match(sp, o->vary))
                         break;
                 o->refcnt--;
@@ -255 +254 @@
                 free(o->http.ws->s);
 
+        if (o->vary != NULL)
+                free(o->vary);
+
         HSH_Freestore(o);
         free(o);