fix: bug when empty event headers (#462) (#463)

codpot · web-flow · commit 2b661640c239 · 2021-10-13T20:02:26.000+10:00
Due to in several express.js libraries treats req.headers as NonNullable Object, when event headers is null causing bugs. When incoming headers from API Gateway is null, it should transform to {}. Fix #462
diff --git a/src/event-sources/utils.js b/src/event-sources/utils.js
@@ -32,7 +32,7 @@ function getRequestValuesFromEvent ({
 
   if (event.multiValueHeaders) {
     headers = getCommaDelimitedHeaders({ headersMap: event.multiValueHeaders, lowerCaseKey: true })
-  } else {
+  } else if (event.headers) {
     headers = event.headers
   }
 

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ function getRequestValuesFromEvent ({`
`32`	`32`
`33`	`33`	`if (event.multiValueHeaders) {`
`34`	`34`	`headers = getCommaDelimitedHeaders({ headersMap: event.multiValueHeaders, lowerCaseKey: true })`
`35`		`- } else {`
	`35`	`+ } else if (event.headers) {`
`36`	`36`	`headers = event.headers`
`37`	`37`	`}`
`38`	`38`