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

test427: verify sending more cookies than fit in a 8190 bytes line #11303

Closed
wants to merge 3 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jun 12, 2023

curl will then only populate the header with cookies that fit, dropping
ones that otherwise would have been sent
Since this option might set multiple cookies in the same line, it does
not make total sense to cap this at 4096 bytes, which is the limit for a
single cookie name or value.
@bagder bagder closed this in 1899899 Jun 12, 2023
bagder added a commit that referenced this pull request Jun 12, 2023
curl will then only populate the header with cookies that fit, dropping
ones that otherwise would have been sent

Ref: https://curl.se/mail/lib-2023-06/0020.html

Closes #11303
bagder added a commit that referenced this pull request Jun 12, 2023
Since this option might set multiple cookies in the same line, it does
not make total sense to cap this at 4096 bytes, which is the limit for a
single cookie name or value.

Closes #11303
@bagder bagder deleted the bagder/test427-largecookies branch June 12, 2023 12:16
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
curl will then only populate the header with cookies that fit, dropping
ones that otherwise would have been sent

Ref: https://curl.se/mail/lib-2023-06/0020.html

Closes curl#11303
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
Since this option might set multiple cookies in the same line, it does
not make total sense to cap this at 4096 bytes, which is the limit for a
single cookie name or value.

Closes curl#11303
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
curl will then only populate the header with cookies that fit, dropping
ones that otherwise would have been sent

Ref: https://curl.se/mail/lib-2023-06/0020.html

Closes curl#11303
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
Since this option might set multiple cookies in the same line, it does
not make total sense to cap this at 4096 bytes, which is the limit for a
single cookie name or value.

Closes curl#11303
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant