Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@remix-run/architect duplicates cookies for Function URLs #8450

Open
THOUSAND-SKY opened this issue Jan 8, 2024 · 0 comments
Open

@remix-run/architect duplicates cookies for Function URLs #8450

THOUSAND-SKY opened this issue Jan 8, 2024 · 0 comments

Comments

@THOUSAND-SKY
Copy link

Reproduction

I believe a fix like this wingleung/remix-aws#7 is also needed for the @remix-run/architect package, for it to work with Function URLs proper. The problem is that function URLs get cookies in both the headers and the cookies array.

An actual reproduce is quite a bit of work for me. However, if you make a lambda via the AWS console and curl it with some cookies, log the event, you'll see the cookies in both headers.cookie and cookies.

I can open a PR if that's a good enough fix for you folks.

System Info

Not relevant

Used Package Manager

pnpm

Expected Behavior

Cookies just once in lambdas behind AWS Lambda on Function URLs.

Actual Behavior

Cookies are seen twice, e.g. _remix_session=something, _remix_session=something and it breaks remix-auth for example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants