Explanation

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3045 d4fa192b-c00b-0410-8231-f00ffab90ce4

diff --git a/varnish-cache/bin/varnishd/rfc2616.c b/varnish-cache/bin/varnishd/rfc2616.c
index 961e7f9cbbaa4f440ed96cd08b5ce1422280c611..1f4383b6e87eef9b77742df51e10e3895d5bf11a 100644 (file)
--- a/varnish-cache/bin/varnishd/rfc2616.c
+++ b/varnish-cache/bin/varnishd/rfc2616.c
@@ -159,6 +159,14 @@ RFC2616_Ttl(const struct sess *sp, const struct http *hp, struct object *obj)
        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)
 {