Skip to content

Commit

Permalink
fix(aws-lambda): set cookies with comma is bugged (#3084)
Browse files Browse the repository at this point in the history
* fix(aws-lambda): setting cookies with comma is bugged

* style: apply lint & format fixes
  • Loading branch information
NamesMT authored Jul 4, 2024
1 parent 75c314c commit 1e3e58b
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/adapter/aws-lambda/handler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -258,7 +258,12 @@ abstract class EventProcessor<E extends LambdaEvent> {

setCookies(event: E, res: Response, result: APIGatewayProxyResult) {
if (res.headers.has('set-cookie')) {
const cookies = res.headers.get('set-cookie')?.split(', ')
const cookies = res.headers.getSetCookie
? res.headers.getSetCookie()
: Array.from(res.headers.entries())
.filter(([k]) => k === 'set-cookie')
.map(([, v]) => v)

if (Array.isArray(cookies)) {
this.setCookiesToResult(event, result, cookies)
res.headers.delete('set-cookie')
Expand Down

0 comments on commit 1e3e58b

Please sign in to comment.