Merge pull request #38 from bigbigDreamer/fix/ISSUE-35

Fix: Expect return true immediately if If-None-Match matches the ETag header (#35)

Author: jonchurch

HISTORY.md

@@ -2,6 +2,7 @@ unreleased
 ==========
 
   * Drop support for Node.js below 0.8
+  * Fix: Ignore `If-Modified-Since` in the presence of `If-None-Match`, according to [spec](https://www.rfc-editor.org/rfc/rfc9110.html#section-13.1.3-5). Fixes [#35](https://github.com/jshttp/fresh/issues/35)
 
 0.5.2 / 2017-09-13
 ==================

index.js

@@ -48,27 +48,26 @@ function fresh (reqHeaders, resHeaders) {
     return false
   }
 
-  // if-none-match
-  if (noneMatch && noneMatch !== '*') {
+  // if-none-match takes precedent over if-modified-since
+  if (noneMatch) {
+    if (noneMatch === '*') {
+      return true
+    }
     var etag = resHeaders.etag
 
     if (!etag) {
       return false
     }
 
-    var etagStale = true
     var matches = parseTokenList(noneMatch)
     for (var i = 0; i < matches.length; i++) {
       var match = matches[i]
       if (match === etag || match === 'W/' + etag || 'W/' + match === etag) {
-        etagStale = false
-        break
+        return true
       }
     }
 
-    if (etagStale) {
-      return false
-    }
+    return false
   }
 
   // if-modified-since

test/fresh.js

@@ -139,10 +139,10 @@ describe('fresh(reqHeaders, resHeaders)', function () {
     })
 
     describe('when only ETag matches', function () {
-      it('should be stale', function () {
+      it('should be fresh', function () {
         var reqHeaders = { 'if-none-match': '"foo"', 'if-modified-since': 'Sat, 01 Jan 2000 00:00:00 GMT' }
         var resHeaders = { etag: '"foo"', 'last-modified': 'Sat, 01 Jan 2000 01:00:00 GMT' }
-        assert.ok(!fresh(reqHeaders, resHeaders))
+        assert.ok(fresh(reqHeaders, resHeaders))
       })
     })