Fix: Check for clean prefix hides access errors

earth-mover · Feb 21, 2025 · 6f0fae2 · 6f0fae2
1 parent 426806b
commit 6f0fae2
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/icechunk/src/storage/mod.rs b/icechunk/src/storage/mod.rs
@@ -406,7 +406,11 @@ pub trait Storage: fmt::Debug + private::Sealed + Sync + Send {
     ) -> StorageResult<DateTime<Utc>>;
 
     async fn root_is_clean(&self) -> StorageResult<bool> {
-        Ok(self.list_objects(&Settings::default(), "").await?.next().await.is_none())
+        match self.list_objects(&Settings::default(), "").await?.next().await {
+            None => Ok(true),
+            Some(Ok(_)) => Ok(false),
+            Some(Err(err)) => Err(err),
+        }
     }
 
     async fn list_chunks(