Fix regression matching multiple ETags in If-None-Match

Author: dougwilson

HISTORY.md

@@ -1,3 +1,8 @@
+unreleased
+==========
+
+  * Fix regression matching multiple ETags in `If-None-Match`
+
 0.5.1 / 2017-09-11
 ==================
 

index.js

@@ -63,13 +63,19 @@ function fresh (reqHeaders, resHeaders) {
       return false
     }
 
+    var etagStale = true
     var matches = noneMatch.split(TOKEN_LIST_REGEXP)
     for (var i = 0; i < matches.length; i++) {
       var match = matches[i]
-      if (match !== etag && match !== 'W/' + etag && 'W/' + match !== etag) {
-        return false
+      if (match === etag || match === 'W/' + etag || 'W/' + match === etag) {
+        etagStale = false
+        break
       }
     }
+
+    if (etagStale) {
+      return false
+    }
   }
 
   // if-modified-since

test/fresh.js

@@ -28,6 +28,14 @@ describe('fresh(reqHeaders, resHeaders)', function () {
       })
     })
 
+    describe('when at least one matches', function () {
+      it('should be fresh', function () {
+        var reqHeaders = { 'if-none-match': ' "bar" , "foo"' }
+        var resHeaders = { 'etag': '"foo"' }
+        assert.ok(fresh(reqHeaders, resHeaders))
+      })
+    })
+
     describe('when etag is missing', function () {
       it('should be stale', function () {
         var reqHeaders = { 'if-none-match': '"foo"' }