give structured context for registry fetch failure

Author: Skgland

src/crates/registry.rs

@@ -1,6 +1,6 @@
 use super::CrateTrait;
 use crate::{PrepareError, Workspace};
-use anyhow::Context as _;
+use anyhow::{anyhow, Context as _};
 use flate2::read::GzDecoder;
 use log::info;
 use std::fs::File;
@@ -163,12 +163,22 @@ impl CrateTrait for RegistryCrate {
             std::fs::create_dir_all(parent)?;
         }
 
-        workspace
+        match workspace
             .http_client()
             .get(self.fetch_url(workspace)?)
             .send()?
-            .error_for_status()?
-            .write_to(&mut BufWriter::new(File::create(&local)?))?;
+            .error_for_status()
+        {
+            Ok(resonse) => resonse,
+            Err(err) => {
+                return Err(anyhow!(err).context(PrepareError::RegistryOperationFailed {
+                    action: "fetch",
+                    krate: self.name.clone(),
+                    version: self.version.clone(),
+                }))
+            }
+        }
+        .write_to(&mut BufWriter::new(File::create(&local)?))?;
 
         Ok(())
     }
@@ -194,7 +204,8 @@ impl CrateTrait for RegistryCrate {
         );
         if let Err(err) = unpack_without_first_dir(&mut tar, dest) {
             let _ = crate::utils::remove_dir_all(dest);
-            Err(err.context(PrepareError::UnpackFailed {
+            Err(err.context(PrepareError::RegistryOperationFailed {
+                action: "unpack",
                 krate: self.name.clone(),
                 version: self.version.clone(),
             }))

src/prepare.rs

@@ -424,12 +424,14 @@ pub enum PrepareError {
         /// repo url
         url: String,
     },
-    /// failed to unpack crate
-    #[error("failed to unpack {krate} version {version}")]
-    UnpackFailed {
-        /// the crates name
+    /// failed to fetch/unpack a crate from a registry
+    #[error("failed to {action} {krate} version {version}")]
+    RegistryOperationFailed {
+        /// the action that failed
+        action: &'static str,
+        /// the name of the crate
         krate: String,
-        /// the crates version
+        /// the version of the crate
         version: String,
     },
 }