CodeGenieApp / CodeGenieApp/serverless-express
[Bug] API Gateway v2 cookies response should be array
- Langage dominant
- JavaScript
- Étoiles
- 5.3k
- Forks
- 676
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
According to API Gateway v2 payload 2.0 docs (https://docs.aws.amazon.com/apigateway/latest/developerguide/http-api-develop-integrations-lambda.html) the 'cookies' should be array.
And below code is current handling logic for it.
https://github.com/vendia/serverless-express/blob/671f914429e9708bfb1ee6ba76b5947a917592df/src/event-sources/aws/api-gateway-v2.js#L65-L67
However, with the latest express (v4.17.1).
When user set cookie using `res.cookie(name, value)`, it set 'set-cookie' header as string at first time.
and convert it as array when using `res.cookie(name, value)` more than once.
So, we should insert Array check logic, and convert it to array when it is not.
Plus,
If we didn't delete 'set-cookie' header in response, API gateway send 'set-cookie' header twice.
So, for gateway v2 and payload 2.0 we should delete 'set-cookie' header field.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez dans src/event-sources/aws/api-gateway-v2.js, au niveau de la logique de traitement liée, et comparez le comportement de la réponse lorsque Express définit un cookie par rapport à plusieurs cookies. Le travail est terminé lorsque API Gateway v2 reçoit les cookies sous forme de tableau et que l’en-tête set-cookie d’origine est supprimé afin qu’il ne soit pas envoyé deux fois.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- aws, express, javascript
- Domaine
- api, backend, cloud
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 45/100