Skip to content

Commit b701d51

Browse files
breninghamfwang
andauthored
fix: Safecheck Object.entries for APIGatewayProxyEventHeaders (#93)
* fix: check if the event.multiValueHeaders exists before attempting to iterate it * fix: checks if event.headers exists before attempting to iterate it. * server: handle API Gateway REST API event without multiValueHeaders --------- Co-authored-by: Frank <[email protected]>
1 parent e330412 commit b701d51

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"open-next": patch
3+
---
4+
5+
server: handle API Gateway REST API event without multiValueHeaders

packages/open-next/src/adapters/event-mapper.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -260,12 +260,12 @@ function normalizeAPIGatewayProxyEventHeaders(
260260
event.multiValueHeaders;
261261
const headers: Record<string, string> = {};
262262

263-
for (const [key, values] of Object.entries(event.multiValueHeaders)) {
263+
for (const [key, values] of Object.entries(event.multiValueHeaders || {})) {
264264
if (values) {
265265
headers[key.toLowerCase()] = values.join(",");
266266
}
267267
}
268-
for (const [key, value] of Object.entries(event.headers)) {
268+
for (const [key, value] of Object.entries(event.headers || {})) {
269269
if (value) {
270270
headers[key.toLowerCase()] = value;
271271
}

0 commit comments

Comments
 (0)