fix: filter out undefinded/null values from query params

Author: Sebastian Sauerer

src/openapi-lambda-adapters.test.ts

@@ -178,6 +178,32 @@ describe('Adapt axios request/response to AWS Lambda Proxy Event/Response', () =
       expect(event.headers['x-null']).toBeUndefined()
       expect(event.headers['x-undefined']).toBeUndefined()
     })
+
+    it('ignores query params with undefined/null values', () => {
+      // given
+      const axiosConfig: AxiosRequestConfig = {
+        method: 'get',
+        url: '/v1/users',
+        params: {
+          limit: 20,
+          offset: null,
+          flag: undefined
+        }
+      }
+      const operation: Operation = {
+        path: '/v1/users',
+        method: HttpMethod.Get,
+        responses: {}
+      }
+
+      // then
+      const event = convertAxiosToApiGw(axiosConfig, operation)
+      expect(event.pathParameters).toEqual({})
+      expect(event.queryStringParameters).toEqual({
+        limit: '20'
+      })
+      expect(event.rawQueryString).toEqual('limit=20')
+    })
   })
 
   describe('Api GW Proxy Response to Axios Response', () => {

src/openapi-lambda-adapters.ts

@@ -41,13 +41,13 @@ export const convertAxiosToApiGw = (config: AxiosRequestConfig, operation: Opera
   }
 
   // extract query params -> convert each value to ta string
-  const queryParams = Object.entries(config.params ?? {}).reduce<APIGatewayProxyEventQueryStringParameters>((queryParams, [key, val]) => {
-    queryParams[key] = val?.toString()
+  const queryParams = Object.entries(config.params ?? {}).filter(([_key, val]) => val !== null && val !== undefined).reduce<APIGatewayProxyEventQueryStringParameters>((queryParams, [key, val]) => {
+    queryParams[key] = val.toString()
     return queryParams
   }, {})
 
   const queryString: string[] = []
-  Object.entries(config.params ?? {}).forEach(([key, val]) => {
+  Object.entries(config.params ?? {}).filter(([_key, val]) => val !== null && val !== undefined).forEach(([key, val]) => {
     if (val && Array.isArray(val)) {
       // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
       queryString.push(...val.map((entry) => `${key}=${entry.toString()}`))
@@ -56,9 +56,6 @@ export const convertAxiosToApiGw = (config: AxiosRequestConfig, operation: Opera
     }
   })
 
-  const urlSearchParams = new URLSearchParams()
-  Object.entries(config.params ?? {}).forEach(([key, val]) => urlSearchParams.append(key, val.toString()))
-
   const headers: Record<string, string> = {}
   for (const [key, val] of Object.entries(config.headers ?? {}).filter(([_key, val]) => val !== null && val !== undefined)) {
     headers[key] = val.toString()