fix: return error instead of panic when fail to upload artifact

succinctlabs · Dec 24, 2024 · e18c4de · e18c4de
1 parent 445e5e6
commit e18c4de
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/crates/sdk/src/network/client.rs b/crates/sdk/src/network/client.rs
@@ -261,9 +261,9 @@ impl NetworkClient {
             self.http.put(&presigned_url).body(bincode::serialize::<T>(item)?).send().await?;
 
         if !response.status().is_success() {
-            log::debug!("Artifact upload failed with status: {}", response.status());
+            log::error!("Artifact upload failed with status: {}", response.status());
+            return Err(anyhow::anyhow!("failed to upload artifact: HTTP {}", response.status()));
         }
-        assert!(response.status().is_success());
 
         Ok(uri)
     }