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

Bump 100-expect length limit #193

Open
daurnimator opened this issue Aug 2, 2021 · 2 comments
Open

Bump 100-expect length limit #193

daurnimator opened this issue Aug 2, 2021 · 2 comments

Comments

@daurnimator
Copy link
Owner

daurnimator commented Aug 2, 2021

https://daniel.haxx.se/blog/2020/02/27/expect-tweaks-in-curl/

Starting in curl 7.69.0 (due to ship on March 4, 2020) we catch up a little with the world around us and now libcurl will only automatically set the Expect: header if the amount of data to send in the body is larger than 1 megabyte. Yes, we raise the limit by 1024 times.

if not length or length > 1024 then

@sudojarvis
Copy link

I would like work this issue. Could you please give some head start.

@daurnimator
Copy link
Owner Author

I think it is likely just changing the constant in the line I linked + writing a test case

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