rustls / futures-rustls

Apache License 2.0
13 stars 7 forks source link

Make it possible to use futures-rustls without ring #7

Closed joshtriplett closed 7 months ago

joshtriplett commented 7 months ago

Add a default-enabled feature ring that passes throuh to rustls/ring. This preserves compatibility with existing users of futures-rustls. Disabling this feature allows using futures-rustls with other crypto providers.

Add a non-default aws-lc-rs feature, as well, for convenience of users who depend on futures-rustls but don't need to depend on rustls directly, so they don't have to add a rustls dependency just to set the feature.

The testsuite currently still requires ring to be enabled, as it calls functions like ClientConfig::builder().

quininer commented 7 months ago

Thank you!