wingleung / remix-aws

AWS adapter for Remix
https://www.npmjs.com/package/remix-aws
MIT License
21 stars 6 forks source link

fix(api-gateway-v2): remove duplicate cookies for function URLs #7

Open THOUSAND-SKY opened 5 months ago

THOUSAND-SKY commented 5 months ago

Function URLs get cookies twice, in headers and the cookies array, which was causing them to duplicate. This will overwrite rather than duplicate.

Ref: https://github.com/awslabs/aws-lambda-rust-runtime/pull/586

Function URLs use the same apigatewayv2 payload otherwise.

THOUSAND-SKY commented 2 months ago

Today function my remix app behind a function URL broke for me suddenly. It seems to me that AWS started escaping the rawQueryString. I don't know why they'd suddenly change stuff like that.

I added another commit here just for visibility; I am not sure the cookie one is needed anymore (but doesn't hurt).