pref: ignore empty http tokens

Author: dougwilson

HISTORY.md

@@ -1,6 +1,7 @@
 unreleased
 ==========
 
+  * pref: ignore empty http tokens
   * deps: http-errors@~1.8.1
     - deps: inherits@2.0.4
     - deps: toidentifier@1.0.1

index.js

@@ -1096,7 +1096,9 @@ function parseTokenList (str) {
         }
         break
       case 0x2c: /* , */
-        list.push(str.substring(start, end))
+        if (start !== end) {
+          list.push(str.substring(start, end))
+        }
         start = end = i + 1
         break
       default:
@@ -1106,7 +1108,9 @@ function parseTokenList (str) {
   }
 
   // final token
-  list.push(str.substring(start, end))
+  if (start !== end) {
+    list.push(str.substring(start, end))
+  }
 
   return list
 }

test/send.js

@@ -451,7 +451,7 @@ describe('send(file).pipe(res)', function () {
       it('should respond with 412 when ETag unmatched', function (done) {
         request(app)
           .get('/name.txt')
-          .set('If-Match', ' "foo", "bar" ')
+          .set('If-Match', ' "foo",, "bar" ,')
           .expect(412, done)
       })