r3045 - trunk/varnish-cache/bin/varnishd

phk at projects.linpro.no phk at projects.linpro.no
Thu Jul 31 14:53:41 CEST 2008


Author: phk
Date: 2008-07-31 14:53:40 +0200 (Thu, 31 Jul 2008)
New Revision: 3045

Modified:
   trunk/varnish-cache/bin/varnishd/rfc2616.c
Log:
Explanation



Modified: trunk/varnish-cache/bin/varnishd/rfc2616.c
===================================================================
--- trunk/varnish-cache/bin/varnishd/rfc2616.c	2008-07-31 12:50:36 UTC (rev 3044)
+++ trunk/varnish-cache/bin/varnishd/rfc2616.c	2008-07-31 12:53:40 UTC (rev 3045)
@@ -159,6 +159,14 @@
 	return (ttd);
 }
 
+/*
+ * We could move this policy to vcl_fetch{} now but I have decided to leave
+ * it here for the POLA principle.  It is not credible to think that a
+ * majority of our uses will change the cacheability decision, so moving
+ * it to VCL would just make the average and median vcl_fetch{} implementation
+ * harder for people to write.  Instead the minority who want to override
+ * the RFC2616 mandated behaviour, can do so in their vcl_fetch{}
+ */
 int
 RFC2616_cache_policy(const struct sess *sp, const struct http *hp)
 {




More information about the varnish-commit mailing list