Add 'native-tls' feature to reqwest dependency#1166
Conversation
|
Tested it on MacOS and it solves my issue. |
jaberjaber23
left a comment
There was a problem hiding this comment.
This adds the native-tls feature to reqwest but no call site actually selects native-tls at runtime. With rustls-tls already enabled, reqwest uses rustls by default. Net effect of this change as-is: ~100KB larger binary, no behavior change.
If the goal is to use the OS certificate store (e.g. corporate proxy with private CA), the change needs a second piece: a runtime select via reqwest::ClientBuilder::use_native_tls() or use_rustls_tls() on the specific clients that need it.
Could you describe the use case driving this and add the runtime select? Otherwise the feature flag is decorative.
|
I don't think that is the case in recent reqwest versions:
|
|
Closing. The change adds the |
Summary
Fixes #1160
Changes
Enables native-tls in reqwest
Testing
cargo clippy --workspace --all-targets -- -D warningspassescargo test --workspacepassesSecurity