Remove parity features

Resolves Azure#2750 and replaces Azure#2811 by removing the `reqwest_rustls` feature which was there only to enable `reqwest/rustls-tls-native-roots-no-provider`. But using resolver v2 and newer, dependents can already do that by just taking a dependency on `reqwest` themselves to add that feature. In some cases, they may need to import `azure_core` with `default-features=false`, but passing a different `reqwest::Client` via our `TransportOptions` is easy.

Not only will this scale better, but should help avoid opinionated feature selection that adversely impacts customers as with Azure#2750.

Author: heaths

Cargo.lock

@@ -117,7 +117,6 @@ quote = "1.0.37"
 rand = "0.9.1"
 rand_chacha = "0.9"
 reqwest = { version = "0.12.23", features = [
-  "json",
   "stream",
 ], default-features = false }
 rust_decimal.version = "1.37.2"

Cargo.toml

@@ -63,7 +63,6 @@ hmac_rust = ["dep:sha2", "dep:hmac"]
 reqwest = ["typespec_client_core/reqwest"]
 reqwest_deflate = ["typespec_client_core/reqwest_deflate"]
 reqwest_gzip = ["typespec_client_core/reqwest_gzip"]
-reqwest_rustls = ["typespec_client_core/reqwest_rustls"]
 test = ["typespec_client_core/test"]
 tokio = ["dep:tokio", "typespec_client_core/tokio"]
 xml = ["typespec_client_core/xml"]
@@ -80,7 +79,6 @@ features = [
   "reqwest",
   "reqwest_deflate",
   "reqwest_gzip",
-  "reqwest_rustls",
   "tokio",
   "xml",
 ]

sdk/core/azure_core/Cargo.toml

@@ -39,17 +39,8 @@ tracing-subscriber.workspace = true
 [features]
 default = ["reqwest"]
 reqwest = ["azure_core/reqwest"]
-reqwest_rustls = ["azure_core/reqwest_rustls"]
 tokio = ["dep:tokio", "azure_core/tokio", "tokio/process"]
 client_certificate = ["openssl"]
 
 [lints]
 workspace = true
-
-[package.metadata.docs.rs]
-features = [
-  "client_certificate",
-  "reqwest",
-  "reqwest_rustls",
-  "tokio",
-]

sdk/identity/azure_identity/Cargo.toml

@@ -64,9 +64,6 @@ json = ["typespec/json"]
 reqwest = ["reqwest/native-tls"]
 reqwest_deflate = ["reqwest/deflate"]
 reqwest_gzip = ["reqwest/gzip"]
-reqwest_rustls = [
-  "reqwest/rustls-tls-native-roots-no-provider",
-] # Remove dependency on banned `ring` crate; requires manually configuring crypto provider.
 test = [] # Enables extra tracing including error bodies that may contain PII.
 tokio = ["tokio/sync", "tokio/time"]
 wasm_bindgen = ["dep:wasm-bindgen-futures", "gloo-timers/futures"]
@@ -89,7 +86,6 @@ features = [
   "reqwest",
   "reqwest_deflate",
   "reqwest_gzip",
-  "reqwest_rustls",
   "tokio",
   "wasm_bindgen",
   "xml",

sdk/typespec/typespec_client_core/Cargo.toml

@@ -3,14 +3,14 @@
 
 //! Built-in HTTP clients.
 
-#[cfg(not(any(feature = "reqwest", feature = "reqwest_rustls")))]
+#[cfg(not(feature = "reqwest"))]
 mod noop;
-#[cfg(any(feature = "reqwest", feature = "reqwest_rustls"))]
+#[cfg(feature = "reqwest")]
 mod reqwest;
 
-#[cfg(not(any(feature = "reqwest", feature = "reqwest_rustls")))]
+#[cfg(not(feature = "reqwest"))]
 use self::noop::new_noop_client;
-#[cfg(any(feature = "reqwest", feature = "reqwest_rustls"))]
+#[cfg(feature = "reqwest")]
 use self::reqwest::new_reqwest_client;
 
 use crate::http::{RawResponse, Request};
@@ -20,11 +20,11 @@ use typespec::error::Result;
 
 /// Create a new [`HttpClient`].
 pub fn new_http_client() -> Arc<dyn HttpClient> {
-    #[cfg(any(feature = "reqwest", feature = "reqwest_rustls"))]
+    #[cfg(feature = "reqwest")]
     {
         new_reqwest_client()
     }
-    #[cfg(not(any(feature = "reqwest", feature = "reqwest_rustls")))]
+    #[cfg(not(feature = "reqwest"))]
     {
         new_noop_client()
     }