Merge pull request #1185 from ScoobySmack2612/HTN/handle-canonical-sa…

…nitized-headers

Pkg: httpHeaderNormalizer to handle canonical sanitized headers

packages/http-header-normalizer/__tests__/index.js

@@ -285,3 +285,26 @@ test('It should not fail if the event does not contain headers', async (t) => {
   t.deepEqual(resultingEvent, expectedEvent)
   t.deepEqual(resultingEvent.rawHeaders, undefined)
 })
+
+test('It should not fail given a corrupted header key', async (t) => {
+  const handler = middy((event, context) => event)
+
+  handler.use(httpHeaderNormalizer({ canonical: true }))
+
+  const event = {
+    headers: {
+      'X----': 'foo'
+    }
+  }
+
+  const expectedHeaders = {
+    'X----': 'foo'
+  }
+
+  const originalHeaders = { ...event.headers }
+
+  const resultingEvent = await handler(event, context)
+
+  t.deepEqual(resultingEvent.headers, expectedHeaders)
+  t.deepEqual(resultingEvent.rawHeaders, originalHeaders)
+})

packages/http-header-normalizer/index.js

@@ -46,7 +46,7 @@ const normalizeHeaderKey = (key, canonical) => {
 
   return lowerCaseKey
     .split('-')
-    .map((text) => text[0].toUpperCase() + text.substr(1))
+    .map((text) => (text[0] || '').toUpperCase() + text.substr(1))
     .join('-')
 }