Another rustls fix

Author: sbernauer

Cargo.lock

@@ -34,7 +34,8 @@ ldap3 = { version = "0.12", features = ["gssapi", "tls"] }
 moka = { version = "0.12", features = ["future"] }
 native-tls = "0.2.12"
 pin-project = "1.1"
-reqwest = { version = "0.13", default-features = false, features = ["json", "form", "query", "rustls-no-provider"] }
+reqwest = { version = "0.13", features = ["json", "form", "query"] }
+rustls = "0.23"
 rustls-pki-types = "1.13"
 semver = "1.0"
 serde = { version = "1.0", features = ["derive"] }

Cargo.toml

@@ -19,6 +19,7 @@ clap.workspace = true
 flate2.workspace = true
 futures.workspace = true
 hyper.workspace = true
+rustls.workspace = true
 snafu.workspace = true
 tar.workspace = true
 tokio.workspace = true

rust/bundle-builder/Cargo.toml

@@ -85,6 +85,13 @@ enum StartupError {
 async fn main() -> Result<(), StartupError> {
     let args = Args::parse();
 
+    // As stackable-operator pulls in ring and reqwest >= 0.13 pulls in aws_lc_rs, we need
+    // to explicitly tell rustls what provider to use. As other operators use ring, we use
+    // that for consistency reasons here as well.
+    rustls::crypto::ring::default_provider()
+        .install_default()
+        .expect("failed to install ring rustls provider");
+
     // NOTE (@NickLarsenNZ): Before stackable-telemetry was used:
     // - The console log level was set by `OPA_BUNDLE_BUILDER_LOG`, and is now `CONSOLE_LOG` (when using Tracing::pre_configured).
     // - The file log level was set by `OPA_BUNDLE_BUILDER_LOG`, and is now set via `FILE_LOG` (when using Tracing::pre_configured).

rust/bundle-builder/src/main.rs

@@ -19,6 +19,7 @@ fnv.workspace = true
 futures.workspace = true
 indoc.workspace = true
 pin-project.workspace = true
+rustls.workspace = true
 semver.workspace = true
 serde_json.workspace = true
 serde.workspace = true