change: separate out tls feature flags

A successor to #53, this separates out the tls feature flags, which should be nicer.

Fixes: #54

Cargo.toml

@@ -20,15 +20,18 @@ features = ["docs"]
 rustdoc-args = ["--cfg", "feature=\"docs\""]
 
 [features]
-default = ["h1_client"]
+default = ["h1_client", "native-tls"]
 docs = ["h1_client", "curl_client", "wasm_client", "hyper_client"]
-h1_client = ["async-h1", "async-std", "async-native-tls"]
-h1_client_rustls = ["async-h1", "async-std", "async-tls"]
+
+h1_client = ["async-h1", "async-std"]
 native_client = ["curl_client", "wasm_client"]
 curl_client = ["isahc", "async-std"]
 wasm_client = ["js-sys", "web-sys", "wasm-bindgen", "wasm-bindgen-futures", "futures"]
 hyper_client = ["hyper", "hyper-tls", "http-types/hyperium_http", "futures-util"]
 
+native-tls = ["async-native-tls"]
+rustls = ["async-tls"]
+
 [dependencies]
 async-trait = "0.1.37"
 http-types = "2.3.0"

src/h1.rs

@@ -74,15 +74,15 @@ impl HttpClient for H1Client {
     }
 }
 
-#[cfg(not(feature = "h1_client_rustls"))]
+#[cfg(all(feature = "h1_client", feature = "native-tls"))]
 async fn add_tls(
     host: String,
     stream: async_std::net::TcpStream,
 ) -> Result<async_native_tls::TlsStream<async_std::net::TcpStream>, async_native_tls::Error> {
     async_native_tls::connect(host, stream).await
 }
 
-#[cfg(feature = "h1_client_rustls")]
+#[cfg(all(feature = "h1_client", feature = "rustls"))]
 async fn add_tls(
     host: String,
     stream: async_std::net::TcpStream,