Merge pull request #450 from Prefinem/clean-headers

fixing when headers are undefined
Raxvis · Dec 1, 2021 · 75578a7 · 75578a7
2 parents 122aba7 + 89b3fef
commit 75578a7
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -52,5 +52,5 @@
     "staged": "lint-staged",
     "test": "./node_modules/.bin/jest"
   },
-  "version": "4.4.1"
+  "version": "4.4.2"
 }
diff --git a/src/server/proxyEvent.js b/src/server/proxyEvent.js
@@ -4,7 +4,7 @@ const handleRequestResponse = require('./handleRequestResponse');
 const http = require('http');
 const url = require('url');
 
-const cleanHeaders = (headers) =>
+const cleanHeaders = (headers = {}) =>
   Object.keys(headers)
     .filter((header) => typeof headers[header] !== 'undefined')
     .reduce((result, header) => ({ ...result, [header]: headers[header] }), {});

diff --git a/tests/server.test.js b/tests/server.test.js
@@ -55,3 +55,16 @@ test('iframe sends undefined cookie', async () => {
 
   await expect(response.body).toEqual('SGVsbG8gV29ybGQh');
 });
+
+test('no headers sent', async () => {
+  const app = express();
+  const noHeaderEvent = { ...event };
+
+  noHeaderEvent.headers = undefined;
+
+  app.get('/', (req, res) => res.send('Hello World!'));
+
+  const response = await lambdaServer(app, true)(noHeaderEvent);
+
+  await expect(response.body).toEqual('SGVsbG8gV29ybGQh');
+});