fix: if disk has no enough space, client return failed (#973)

Signed-off-by: Gaius <[email protected]>

Author: gaius-qi

Cargo.lock

@@ -12,7 +12,7 @@ members = [
 ]
 
 [workspace.package]
-version = "0.2.7"
+version = "0.2.8"
 authors = ["The Dragonfly Developers"]
 homepage = "https://d7y.io/"
 repository = "https://github.com/dragonflyoss/client.git"
@@ -22,13 +22,13 @@ readme = "README.md"
 edition = "2021"
 
 [workspace.dependencies]
-dragonfly-client = { path = "dragonfly-client", version = "0.2.7" }
-dragonfly-client-core = { path = "dragonfly-client-core", version = "0.2.7" }
-dragonfly-client-config = { path = "dragonfly-client-config", version = "0.2.7" }
-dragonfly-client-storage = { path = "dragonfly-client-storage", version = "0.2.7" }
-dragonfly-client-backend = { path = "dragonfly-client-backend", version = "0.2.7" }
-dragonfly-client-util = { path = "dragonfly-client-util", version = "0.2.7" }
-dragonfly-client-init = { path = "dragonfly-client-init", version = "0.2.7" }
+dragonfly-client = { path = "dragonfly-client", version = "0.2.8" }
+dragonfly-client-core = { path = "dragonfly-client-core", version = "0.2.8" }
+dragonfly-client-config = { path = "dragonfly-client-config", version = "0.2.8" }
+dragonfly-client-storage = { path = "dragonfly-client-storage", version = "0.2.8" }
+dragonfly-client-backend = { path = "dragonfly-client-backend", version = "0.2.8" }
+dragonfly-client-util = { path = "dragonfly-client-util", version = "0.2.8" }
+dragonfly-client-init = { path = "dragonfly-client-init", version = "0.2.8" }
 thiserror = "1.0"
 dragonfly-api = "=2.1.23"
 reqwest = { version = "0.12.4", features = [

Cargo.toml

@@ -206,16 +206,13 @@ impl Task {
             content_length,
         );
 
-        // If the task is not found, check if the storage has enough space to
+        // If the task is not finished, check if the storage has enough space to
         // store the task.
-        if let Ok(None) = self.get(id) {
-            let has_enough_space = self.storage.has_enough_space(content_length)?;
-            if !has_enough_space {
-                return Err(Error::NoSpace(format!(
-                    "not enough space to store the persistent cache task: content_length={}",
-                    content_length
-                )));
-            }
+        if !task.is_finished() && !self.storage.has_enough_space(content_length)? {
+            return Err(Error::NoSpace(format!(
+                "not enough space to store the persistent cache task: content_length={}",
+                content_length
+            )));
         }
 
         self.storage.download_task_started(