fix(parser): set APIGatewayProxyEventSchema body and query string key…

…s to be nullable (#2465) Co-authored-by: 1482568 <austin.blythe@hyatt.com>
aws-powertools · May 2, 2024 · 7ce5b3c · 7ce5b3c
1 parent 1f6cc41
commit 7ce5b3c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/packages/parser/src/schemas/apigw.ts b/packages/parser/src/schemas/apigw.ts
@@ -97,14 +97,14 @@ const APIGatewayProxyEventSchema = z.object({
     'OPTIONS',
   ]),
   headers: z.record(z.string()).optional(),
-  queryStringParameters: z.record(z.string()).optional(),
+  queryStringParameters: z.record(z.string()).nullable(),
   multiValueHeaders: z.record(z.array(z.string())).optional(),
-  multiValueQueryStringParameters: z.record(z.array(z.string())).optional(),
+  multiValueQueryStringParameters: z.record(z.array(z.string())).nullable(),
   requestContext: APIGatewayEventRequestContext,
   pathParameters: z.record(z.string()).optional().nullish(),
   stageVariables: z.record(z.string()).optional().nullish(),
   isBase64Encoded: z.boolean().optional(),
-  body: z.string().optional(),
+  body: z.string().nullable(),
 });
 
 export { APIGatewayProxyEventSchema, APIGatewayCert };
diff --git a/packages/parser/tests/events/apiGatewayAuthorizerRequestEvent.json b/packages/parser/tests/events/apiGatewayAuthorizerRequestEvent.json
@@ -20,6 +20,9 @@
   "queryStringParameters": {
     "QueryString1": "queryValue1"
   },
+  "multiValueQueryStringParameters": {
+    "QueryString1": ["queryValue1"]
+  },
   "pathParameters": {},
   "stageVariables": {
     "StageVar1": "stageValue1"
@@ -64,5 +67,6 @@
     "resourceId": "ANY /request",
     "resourcePath": "/request",
     "stage": "test"
-  }
+  },
+  "body": null
 }
diff --git a/packages/parser/tests/unit/envelopes/apigwt.test.ts b/packages/parser/tests/unit/envelopes/apigwt.test.ts
@@ -24,7 +24,7 @@ describe('ApigwEnvelope ', () => {
 
     it('should throw no body provided', () => {
       const testEvent = TestEvents.apiGatewayProxyEvent as APIGatewayProxyEvent;
-      testEvent.body = undefined;
+      testEvent.body = null;
 
       expect(() => ApiGatewayEnvelope.parse(testEvent, TestSchema)).toThrow(
         ParseError
@@ -56,7 +56,7 @@ describe('ApigwEnvelope ', () => {
 
     it('should return success false with original body if no body provided', () => {
       const testEvent = TestEvents.apiGatewayProxyEvent as APIGatewayProxyEvent;
-      testEvent.body = undefined;
+      testEvent.body = null;
 
       const resp = ApiGatewayEnvelope.safeParse(testEvent, TestSchema);
       expect(resp).toEqual({