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

OpenSSL requirement is onerous (use native TLS instead) #1

Closed
chris-morgan opened this issue Jan 2, 2017 · 3 comments
Closed

OpenSSL requirement is onerous (use native TLS instead) #1

chris-morgan opened this issue Jan 2, 2017 · 3 comments

Comments

@chris-morgan
Copy link

Switching from reqwest to hyper had the effect of introducing an OpenSSL requirement on Windows and macOS that wasn’t there before, and getting OpenSSL working properly on Windows is a nuisance. reqwest does what a normal person would expect: it uses the platform’s native TLS implementation, via the native-tls crate.

It’d be good for this crate to switch back to using native-tls so that it can be used on Windows without having to have OpenSSL set up on platforms where it’s tricky to do so.

@rushmorem
Copy link
Owner

I would love to switch back to reqwest. The only reason I switched to hyper is because it allows me to set_read_timeout and set_write_timeout. I was running into an issue where a request would hang for a long time if the connection was bad. If there is a way to get around this while using reqwest I will happily switch back.

@rushmorem
Copy link
Owner

rushmorem commented Jan 2, 2017

Alternatively, I can just use native-tls directly since I'm simply performing a GET request.

@rushmorem
Copy link
Owner

I have disabled all Hyper features, which include ssl. We are now using native-tls directly. Thank you for bringing this up. I think a full blown HTTP library was overkill for our use case anyway.

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

No branches or pull requests

2 participants