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

Set CURLOPT_HTTP09_ALLOWED to true #155

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ExpDev07
Copy link

CURL allowed HTTP/0.9 responses by default before 7.66.0, but since 7.66.0, libcurl requires this option set to 1L to allow HTTP/0.9 responses. This is required for interacting with Monero wallet as it uses HTTP/0.9.

https://curl.se/libcurl/c/CURLOPT_HTTP09_ALLOWED.html

@recanman
Copy link
Contributor

This code is old and is to be replaced by #154.

@ExpDev07
Copy link
Author

ExpDev07 commented May 25, 2024

I'll just patch it until the release then @recanman.

But wouldn't this still be good to merge as it's such a small non breaking change, and we don't know when the new version is going to be replaced. If people wanna use it and have a newer cURL version, they literally can't use it without the patch.

@recanman
Copy link
Contributor

Good point. I don't have write access and I cannot merge this.

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

Successfully merging this pull request may close these issues.

2 participants