-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[node] add test for multiple cookies on edge #10746
Conversation
|
I believe this will be fixed in the next release of |
Have you seen #10767 yet? This might have some impact to your work. |
@Kikobeats you skipped the test this PR adds :) It's named |
With the changes from #10712, this test case is now passing.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it 👍
We test this in a different way here: https://github.com/vercel/vercel/blob/main/packages/node/test/dev-fixtures/multiple-cookies.ts Maybe we should do both? Closing for now. |
This PR adds a test case demonstrating the ability to set multiple cookies in an edge runtime API handler.