Date has to be up to date when the Age header is added

kornelski · kornelski · commit 9ce9e12d9c0c · 2018-04-21T14:47:36.000+01:00
diff --git a/index.js b/index.js
@@ -5,7 +5,10 @@ const statusCodeCacheableByDefault = [200, 203, 204, 206, 300, 301, 404, 405, 41
 // This implementation does not understand partial responses (206)
 const understoodStatuses = [200, 203, 204, 300, 301, 302, 303, 307, 308, 404, 405, 410, 414, 501];
 
-const hopByHopHeaders = {'connection':true, 'keep-alive':true, 'proxy-authenticate':true, 'proxy-authorization':true, 'te':true, 'trailer':true, 'transfer-encoding':true, 'upgrade':true};
+const hopByHopHeaders = {
+    'date': true, // included, because we add Age update Date
+    'connection':true, 'keep-alive':true, 'proxy-authenticate':true, 'proxy-authorization':true, 'te':true, 'trailer':true, 'transfer-encoding':true, 'upgrade':true
+};
 const excludedFromRevalidationUpdate = {
     // Since the old body is reused, it doesn't make sense to change properties of the body
     'content-length': true, 'content-encoding': true, 'transfer-encoding': true,
@@ -229,6 +232,7 @@ module.exports = class CachePolicy {
             headers.warning = (headers.warning ? `${headers.warning}, ` : '') + '113 - "rfc7234 5.5.4"';
         }
         headers.age = `${Math.round(age)}`;
+        headers.date = new Date(this.now()).toUTCString();
         return headers;
     }
 
diff --git a/test/responsetest.js b/test/responsetest.js
@@ -372,7 +372,7 @@ describe('Response headers', function() {
         assert(!h.te);
         assert(!h.oompa);
         assert.equal(h['cache-control'], 'public, max-age=333');
-        assert.equal(h.date, 'now', "date must stay the same for expires, age, etc");
+        assert.notEqual(h.date, 'now', "updated age requires updated date");
         assert.equal(h.custom, 'header');
         assert.equal(h.age, '11');
         assert.equal(res.headers.age, '10');
