feat(self_update): add proxy sanity checks

Sometimes, proxies can be incorrectly installed for whatever
reason. This adds a new check after initial component
installation to call all proxies with --version, and to
fail if one of them doesn't report success.

Author: manyinsects

src/cli/self_update.rs

@@ -853,6 +853,33 @@ fn install_proxies_with_opts(process: &Process, force_hard_links: bool) -> Resul
     Ok(())
 }
 
+fn check_proxy_sanity(
+    process: &Process,
+    components: &[&str],
+    toolchain: &ToolchainDesc,
+) -> Result<()> {
+    let bin_path = process.cargo_home()?.join("bin");
+
+    // Sometimes linking a proxy produces an unpredictable result, where the proxy
+    // is in place, but manages to not call rustup correctly. One way to make sure we
+    // don't run headfirst into the wall is to at least try and run our freshly
+    // installed proxies, to see if they return some manner of reasonable output.
+    // We limit ourselves to the most common two installed components (cargo and rustc),
+    // because their binary names also happen to match up, which is not necessarily
+    // a given.
+    for component in components.iter().filter(|c| ["cargo", "rustc"].contains(c)) {
+        let cmd = Command::new(bin_path.join(format!("{}{EXE_SUFFIX}", component)))
+            .args([&format!("+{}", toolchain), "--version"])
+            .status();
+
+        if !cmd.is_ok_and(|status| status.success()) {
+            return Err(RustupError::InvalidProxyInstall.into());
+        }
+    }
+
+    Ok(())
+}
+
 async fn maybe_install_rust(
     current_dir: PathBuf,
     quiet: bool,
@@ -907,6 +934,8 @@ async fn maybe_install_rust(
             .0
         };
 
+        check_proxy_sanity(process, components, desc)?;
+
         cfg.set_default(Some(&desc.into()))?;
         writeln!(process.stdout().lock())?;
         common::show_channel_update(&cfg, PackageUpdate::Toolchain(desc.clone()), Ok(status))?;

src/errors.rs

@@ -141,6 +141,12 @@ pub enum RustupError {
         target: TargetTriple,
         suggestion: Option<String>,
     },
+    #[error(
+        "unable to correctly link proxies\n\
+        this might be a bug in rustup, please open a new issue here:\n\
+        https://github.com/rust-lang/rustup/issues/new"
+    )]
+    InvalidProxyInstall,
     #[error("unknown metadata version: '{0}'")]
     UnknownMetadataVersion(String),
     #[error("manifest version '{0}' is not supported")]