Remove webpki feature dependency

smithy-lang · Jan 5, 2023 · 7285bd9 · 7285bd9
1 parent 7a6ddfb
commit 7285bd9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/rust-runtime/aws-smithy-client/Cargo.toml b/rust-runtime/aws-smithy-client/Cargo.toml
@@ -28,7 +28,7 @@ hyper = { version = "0.14.12", features = ["client", "http2", "http1", "tcp"], o
 # cargo does not support optional test dependencies, so to completely disable rustls when
 # the native-tls feature is enabled, we need to add the webpki-roots feature here.
 # https://github.com/rust-lang/cargo/issues/1596
-hyper-rustls = { version = "0.23.0", optional = true, features = ["rustls-native-certs", "http2", "webpki-roots"] }
+hyper-rustls = { version = "0.23.0", optional = true, features = ["rustls-native-certs", "http2"] }
 hyper-tls = { version = "0.5.0", optional = true }
 lazy_static = { version = "1", optional = true }
 pin-project-lite = "0.2.7"

diff --git a/rust-runtime/aws-smithy-client/src/hyper_ext.rs b/rust-runtime/aws-smithy-client/src/hyper_ext.rs
@@ -58,7 +58,10 @@
 //! use aws_smithy_client::http_connector::ConnectorSettings;
 //!
 //! let https_connector = hyper_rustls::HttpsConnectorBuilder::new()
+//! # /* webpki_roots aren't actually enabled in the dependency so comment this out
 //!     .with_webpki_roots()
+//! # */
+//! #   .with_native_roots()
 //!     .https_only()
 //!     .enable_http1()
 //!     .enable_http2()