header value coerce to array instead of split

Author: kellyselden

src/fastboot-headers.js

@@ -5,7 +5,17 @@ function FastBootHeaders(headers) {
   this.headers = {};
 
   for (var header in headers) {
-    this.headers[header] = headers[header].split(', ');
+    let value = headers[header];
+
+    // Express gives us either a string
+    // or an array of strings if there are multiple values.
+    // We want to support the Header spec
+    // so we will coerce to an array always.
+    if (typeof value === 'string') {
+      value = [value];
+    }
+
+    this.headers[header] = value;
   }
 }
 

test/fastboot-headers-test.js

@@ -8,9 +8,7 @@ var FastBootHeaders = alchemistRequire('fastboot-headers.js');
 describe('FastBootHeaders', function() {
   it('returns an array of header values from getAll, regardless of header name casing', function() {
     var headers = {
-      // Express concatenates repeated keys with ', '
-      // and also lowercases the keys
-      'x-test-header': 'value1, value2'
+      'x-test-header': ['value1', 'value2']
     };
     headers = new FastBootHeaders(headers);
 
@@ -20,9 +18,7 @@ describe('FastBootHeaders', function() {
 
   it('returns an emtpy array when a header is not present', function() {
     var headers = {
-      // Express concatenates repeated keys with ', '
-      // and also lowercases the keys
-      'x-test-header': 'value1, value2'
+      'x-test-header': ['value1', 'value2']
     };
     headers = new FastBootHeaders(headers);
 
@@ -32,9 +28,7 @@ describe('FastBootHeaders', function() {
 
   it('returns the first value when using get, regardless of case', function() {
     var headers = {
-      // Express concatenates repeated keys with ', '
-      // and also lowercases the keys
-      'x-test-header': 'value1, value2'
+      'x-test-header': ['value1', 'value2']
     };
     headers = new FastBootHeaders(headers);
 
@@ -44,9 +38,7 @@ describe('FastBootHeaders', function() {
 
   it('returns null when using get when a header is not present', function() {
     var headers = {
-      // Express concatenates repeated keys with ', '
-      // and also lowercases the keys
-      'x-test-header': 'value1, value2'
+      'x-test-header': ['value1', 'value2']
     };
     headers = new FastBootHeaders(headers);
 
@@ -56,9 +48,7 @@ describe('FastBootHeaders', function() {
 
   it('returns whether or not a header is present via has, regardless of casing', function() {
     var headers = {
-      // Express concatenates repeated keys with ', '
-      // and also lowercases the keys
-      'x-test-header': 'value1, value2'
+      'x-test-header': ['value1', 'value2']
     };
     headers = new FastBootHeaders(headers);
 

test/fastboot-response-test.js

@@ -9,7 +9,7 @@ describe("FastBootResponse", function() {
   beforeEach(function () {
     var mockResponse = {
       _headers: {
-        "i-am-a": "mock header, me too",
+        "i-am-a": ["mock header", "me too"],
         "cookie": ""
       }
     };