-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Optional rustls support #378
Comments
Shouldn't |
For anyone else wondering how to add rust-tls, set feature flags like the following in your Cargo.toml: reqwest = {
version = "0.11.16",
features = ["json", "rustls-tls"],
default-features = false
} |
Using e.g. (using
UPDATE: Ugly logging, yes, but this is good -- it is HTTP 2/at work. It is good that rust-tls made it happen :) |
With the ability to now optionally disable the
default-tls
feature (#375), it would also be fantastic to allow people to use reqwest with a rustls backend instead.rustls
as a dependency,optional = true
. This will automatically promote to arustls
feature, defaulted off.connect
module can likely inline the connector code from hyper-tls and hyper-rustls, so we can remove that dependency.Box<AsyncRead + AsyncWrite>
.default-tls
andrustls
are enabled, the native-tls backend should be chosen by default.ClientBuilder
can gain methods to pick a backend at runtime.The text was updated successfully, but these errors were encountered: