Buy commercial curl support from WolfSSL. We help you work
out your issues, debug your libcurl applications, use the API, port to new
platforms, add new features and more. With a team lead by the curl founder
himself.
Re: Issues with new cookie length limits
- Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]
From: Aleksandar Lazic via curl-library <curl-library_at_lists.haxx.se>
Date: Tue, 6 Jun 2023 22:14:45 +0200
Hi David.
On 2023-06-06 (Di.) 15:01, Chamberlin, David via curl-library wrote:
> Hi,
>
> I work for a large company which has long made grateful use of curl’s
> features both as a command line tool and as a library.
>
> Since curl 8, (I believe) a limit to the maximum size of the cookie
> header has been applied in curl. This has created issues when used with
> our intranet because of cookies automatically added by software over
> which we have little control.
>
> We have patched the curl package (in cookie.h) that we build for
> internal consumption to raise this limit (MAX_COOKIE_HEADER_LEN) from
> 8kB to 32kB which fixes our internal issue.
>
> Would the team be amenable to a pull request that adds an option to
> CMakeLists and/or configure to make this limit configurable at compile time?
>
> We also picked up a bug in that code in that the length limit check
> seems to be calculated incorrectly – we can submit a pull request for
> that also.
The was a length discussion about this topic last month, I'm pretty sure
you will find in that thread your answer :-).
https://curl.se/mail/lib-2023-05/index.html#msg11
> -David
Regards
Alex
Date: Tue, 6 Jun 2023 22:14:45 +0200
Hi David.
On 2023-06-06 (Di.) 15:01, Chamberlin, David via curl-library wrote:
> Hi,
>
> I work for a large company which has long made grateful use of curl’s
> features both as a command line tool and as a library.
>
> Since curl 8, (I believe) a limit to the maximum size of the cookie
> header has been applied in curl. This has created issues when used with
> our intranet because of cookies automatically added by software over
> which we have little control.
>
> We have patched the curl package (in cookie.h) that we build for
> internal consumption to raise this limit (MAX_COOKIE_HEADER_LEN) from
> 8kB to 32kB which fixes our internal issue.
>
> Would the team be amenable to a pull request that adds an option to
> CMakeLists and/or configure to make this limit configurable at compile time?
>
> We also picked up a bug in that code in that the length limit check
> seems to be calculated incorrectly – we can submit a pull request for
> that also.
The was a length discussion about this topic last month, I'm pretty sure
you will find in that thread your answer :-).
https://curl.se/mail/lib-2023-05/index.html#msg11
> -David
Regards
Alex
-- Unsubscribe: https://lists.haxx.se/mailman/listinfo/curl-library Etiquette: https://curl.se/mail/etiquette.htmlReceived on 2023-06-06