Rename default-tls to native-tls

Author: djc

Cargo.toml

@@ -15,14 +15,14 @@ curl-backend = ["download/curl-backend"]
 default = [
     "curl-backend",
     "reqwest-backend",
-    "reqwest-default-tls",
+    "reqwest-native-tls",
     "reqwest-rustls-tls",
 ]
 
 reqwest-backend = ["download/reqwest-backend"]
 vendored-openssl = ['openssl/vendored']
 
-reqwest-default-tls = ["download/reqwest-default-tls"]
+reqwest-native-tls = ["download/reqwest-native-tls"]
 reqwest-rustls-tls = ["download/reqwest-rustls-tls"]
 
 # Include in the default set to disable self-update and uninstall.

ci/run.bash

@@ -8,7 +8,7 @@ rustc -vV
 cargo -vV
 
 
-FEATURES=('--no-default-features' '--features' 'curl-backend,reqwest-backend,reqwest-default-tls')
+FEATURES=('--no-default-features' '--features' 'curl-backend,reqwest-backend,reqwest-native-tls')
 case "$(uname -s)" in
   *NT* ) ;; # Windows NT
   * ) FEATURES+=('--features' 'vendored-openssl') ;;
@@ -38,7 +38,7 @@ target_cargo() {
 target_cargo build
 
 download_pkg_test() {
-  features=('--no-default-features' '--features' 'curl-backend,reqwest-backend,reqwest-default-tls')
+  features=('--no-default-features' '--features' 'curl-backend,reqwest-backend,reqwest-native-tls')
   case "$TARGET" in
     # these platforms aren't supported by ring:
     powerpc* ) ;;

download/Cargo.toml

@@ -5,12 +5,10 @@ edition = "2021"
 license = "MIT OR Apache-2.0"
 
 [features]
-
-default = ["reqwest-backend", "reqwest-rustls-tls", "reqwest-default-tls"]
-
+default = ["reqwest-backend", "reqwest-rustls-tls", "reqwest-native-tls"]
 curl-backend = ["curl"]
 reqwest-backend = ["reqwest", "env_proxy"]
-reqwest-default-tls = ["reqwest/default-tls", "dep:once_cell"]
+reqwest-native-tls = ["reqwest/native-tls", "dep:once_cell"]
 reqwest-rustls-tls = ["reqwest/rustls-tls-native-roots", "dep:once_cell"]
 
 [dependencies]

download/src/lib.rs

@@ -14,7 +14,7 @@ use url::Url;
 #[cfg(feature = "curl-backend")]
 const CURL_USER_AGENT: &str = concat!("rustup/", env!("CARGO_PKG_VERSION"), " (curl)");
 
-#[cfg(feature = "reqwest-default-tls")]
+#[cfg(feature = "reqwest-native-tls")]
 const REQWEST_DEFAULT_TLS_USER_AGENT: &str = concat!(
     "rustup/",
     env!("CARGO_PKG_VERSION"),
@@ -34,7 +34,7 @@ pub enum Backend {
 #[derive(Debug, Copy, Clone)]
 pub enum TlsBackend {
     Rustls,
-    Default,
+    NativeTls,
 }
 
 #[derive(Debug, Copy, Clone)]
@@ -289,15 +289,15 @@ pub mod curl {
 pub mod reqwest_be {
     #[cfg(all(
         not(feature = "reqwest-rustls-tls"),
-        not(feature = "reqwest-default-tls")
+        not(feature = "reqwest-native-tls")
     ))]
     compile_error!("Must select a reqwest TLS backend");
 
     use std::io;
     use std::time::Duration;
 
     use anyhow::{anyhow, Context, Result};
-    #[cfg(any(feature = "reqwest-rustls-tls", feature = "reqwest-default-tls"))]
+    #[cfg(any(feature = "reqwest-rustls-tls", feature = "reqwest-native-tls"))]
     use once_cell::sync::Lazy;
     use reqwest::{header, Client, ClientBuilder, Proxy, Response};
     use tokio_stream::StreamExt;
@@ -367,7 +367,7 @@ pub mod reqwest_be {
         catcher().unwrap()
     });
 
-    #[cfg(feature = "reqwest-default-tls")]
+    #[cfg(feature = "reqwest-native-tls")]
     static CLIENT_DEFAULT_TLS: Lazy<Client> = Lazy::new(|| {
         let catcher = || {
             client_generic()
@@ -400,10 +400,10 @@ pub mod reqwest_be {
             TlsBackend::Rustls => {
                 return Err(DownloadError::BackendUnavailable("reqwest rustls"));
             }
-            #[cfg(feature = "reqwest-default-tls")]
-            TlsBackend::Default => &CLIENT_DEFAULT_TLS,
-            #[cfg(not(feature = "reqwest-default-tls"))]
-            TlsBackend::Default => {
+            #[cfg(feature = "reqwest-native-tls")]
+            TlsBackend::NativeTls => &CLIENT_DEFAULT_TLS,
+            #[cfg(not(feature = "reqwest-native-tls"))]
+            TlsBackend::NativeTls => {
                 return Err(DownloadError::BackendUnavailable("reqwest default TLS"));
             }
         };

download/tests/download-reqwest-resume.rs

@@ -21,7 +21,7 @@ async fn resume_partial_from_file_url() {
 
     let from_url = Url::from_file_path(&from_path).unwrap();
     download_to_path_with_backend(
-        Backend::Reqwest(TlsBackend::Default),
+        Backend::Reqwest(TlsBackend::NativeTls),
         &from_url,
         &target_path,
         true,
@@ -48,7 +48,7 @@ async fn callback_gets_all_data_as_if_the_download_happened_all_at_once() {
     let received_in_callback = Mutex::new(Vec::new());
 
     download_to_path_with_backend(
-        Backend::Reqwest(TlsBackend::Default),
+        Backend::Reqwest(TlsBackend::NativeTls),
         &from_url,
         &target_path,
         true,

src/utils/utils.rs

@@ -229,11 +229,11 @@ async fn download_file_(
         let tls_backend = if use_rustls {
             TlsBackend::Rustls
         } else {
-            #[cfg(feature = "reqwest-default-tls")]
+            #[cfg(feature = "reqwest-native-tls")]
             {
-                TlsBackend::Default
+                TlsBackend::NativeTls
             }
-            #[cfg(not(feature = "reqwest-default-tls"))]
+            #[cfg(not(feature = "reqwest-native-tls"))]
             {
                 TlsBackend::Rustls
             }