Merge pull request #1133 from robertbeal/http-event-normalizer-errors…

…-fix

fix: don't throw error for non http event

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

@@ -7,30 +7,22 @@ const context = {
   getRemainingTimeInMillis: () => 1000
 }
 
-test('It should throw error when invalid version', async (t) => {
+test('It should not throw error when invalid version', async (t) => {
   const event = {
     version: '3.0'
   }
 
   const handler = middy((event) => event).use(httpEventNormalizer())
-  try {
-    await handler(event, context)
-  } catch (e) {
-    t.is(e.message, 'Unknown http event format')
-  }
+  t.notThrows(async () => await handler(event, context))
 })
 
-test('It should do nothing if not HTTP event', async (t) => {
+test('It should not error if not an HTTP event', async (t) => {
   const event = {
     source: 's3'
   }
 
   const handler = middy((event) => event).use(httpEventNormalizer())
-  try {
-    await handler(event, context)
-  } catch (e) {
-    t.is(e.message, 'Unknown http event format')
-  }
+  t.notThrows(async () => await handler(event, context))
 })
 
 test('It should default queryStringParameters with REST API', async (t) => {

packages/http-event-normalizer/index.js

@@ -3,12 +3,6 @@ const httpEventNormalizerMiddleware = () => {
     const { event } = request
 
     const version = pickVersion(event)
-    const isHttpEvent = isVersionHttpEvent[version]?.(event)
-    if (!isHttpEvent) {
-      throw new Error('Unknown http event format', {
-        cause: { package: '@middy/http-event-normalizer' }
-      })
-    }
     // VPC Lattice is an http event, however uses a different notation
     // - query_string_parameters
     // - is_base64_encoded
@@ -35,10 +29,4 @@ const pickVersion = (event) => {
   return event.version ?? (event.method ? 'vpc' : '1.0')
 }
 
-const isVersionHttpEvent = {
-  '1.0': (event) => typeof event.httpMethod !== 'undefined',
-  '2.0': (event) => typeof event.requestContext.http.method !== 'undefined',
-  vpc: (event) => typeof event.method !== 'undefined'
-}
-
 export default httpEventNormalizerMiddleware