Ignore Pragma when Cache-Control is present but blank

kornelski · Feb 9, 2019 · 747c677 · 747c677
1 parent d5c28b7
commit 747c677
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -85,7 +85,7 @@ module.exports = class CachePolicy {
 
         // When the Cache-Control header field is not present in a request, caches MUST consider the no-cache request pragma-directive
         // as having the same effect as if "Cache-Control: no-cache" were present (see Section 5.2.1).
-        if (!res.headers['cache-control'] && /no-cache/.test(res.headers.pragma)) {
+        if (res.headers['cache-control'] == null && /no-cache/.test(res.headers.pragma)) {
             this._rescc['no-cache'] = true;
         }
     }

diff --git a/test/responsetest.js b/test/responsetest.js
@@ -223,6 +223,15 @@ describe('Response headers', function() {
         assert(cache.stale());
     });
 
+    it('blank cache-control and pragma: no-cache', function() {
+        const cache = new CachePolicy(req, {headers:{
+            'cache-control': '',
+            'pragma': 'no-cache',
+            'last-modified': new Date().toGMTString(),
+        }});
+        assert(!cache.stale());
+    });
+
     it('no-store', function() {
         const cache = new CachePolicy(req, {headers:{
             'cache-control': 'no-store, public, max-age=1',