fix null value error for multiValueHeaders

Author: jeremydaly

lib/request.js

@@ -87,7 +87,7 @@ class REQUEST {
 
     // Set the raw headers (normalize multi-values)
     // per https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2
-    this.rawHeaders = this._multiValueSupport ?
+    this.rawHeaders = this._multiValueSupport && this.app._event.multiValueHeaders != null ?
       Object.keys(this.app._event.multiValueHeaders).reduce((headers,key) =>
         Object.assign(headers,{ [key]: UTILS.fromArray(this.app._event.multiValueHeaders[key]) }),{})
       : this.app._event.headers || {}

test/headers.js

@@ -172,6 +172,12 @@ describe('Header Tests:', function() {
       expect(result).to.deep.equal({ multiValueHeaders: { 'content-type': ['application/json'], 'test': ['testVal1','testVal2'] }, statusCode: 200, body: '{"method":"get","status":"ok"}', isBase64Encoded: false })
     }) // end it
 
+    it('Multi-value Header: /testMulti (null header)', async function() {
+      let _event = Object.assign({},event,{ path: '/testMulti', multiValueHeaders: null })
+      let result = await new Promise(r => api.run(_event,{},(e,res) => { r(res) }))
+      expect(result).to.deep.equal({ multiValueHeaders: { 'content-type': ['application/json'], 'test': ['testVal1','testVal2'] }, statusCode: 200, body: '{"method":"get","status":"ok"}', isBase64Encoded: false })
+    }) // end it
+
     it('HTML Convenience Method: /testHTML', async function() {
       let _event = Object.assign({},event,{ path: '/testHTML'})
       let result = await new Promise(r => api.run(_event,{},(e,res) => { r(res) }))