chore: reduce number of clippy warnings when no TLS stack is enabled

Author: ctron

src/tools.rs

@@ -4,7 +4,6 @@
 use std::collections::HashMap;
 use std::path::PathBuf;
 
-use crate::common::check_target_not_found_err;
 use anyhow::{anyhow, bail, ensure, Context, Result};
 use directories::ProjectDirs;
 use futures_util::stream::StreamExt;
@@ -452,17 +451,20 @@ pub async fn cache_dir() -> Result<PathBuf> {
 }
 
 async fn get_http_client(client_options: &HttpClientOptions) -> Result<reqwest::Client> {
-    let mut builder = reqwest::ClientBuilder::new();
+    let builder = reqwest::ClientBuilder::new();
 
     #[cfg(any(feature = "native-tls", feature = "rustls"))]
     let builder = {
-        builder = builder.danger_accept_invalid_certs(client_options.accept_invalid_certificates);
+        let mut builder =
+            builder.danger_accept_invalid_certs(client_options.accept_invalid_certificates);
 
         if let Some(root_certs) = &client_options.root_certificate {
             let cert = tokio::fs::read(root_certs)
                 .await
                 .with_context(|| "Error reading certificate")
-                .map_err(|err| check_target_not_found_err(err, &root_certs.to_string_lossy()))?;
+                .map_err(|err| {
+                    crate::common::check_target_not_found_err(err, &root_certs.to_string_lossy())
+                })?;
 
             builder = builder.add_root_certificate(
                 reqwest::Certificate::from_pem(&cert)