From 20cbacd78598bcddd7391a64c0cb81b68236435e Mon Sep 17 00:00:00 2001
From: NimBold <NimBold@users.noreply.github.com>
Date: Mon, 29 Jun 2026 21:46:49 +0330
Subject: [PATCH] fix(download): recognize aria2 HTTP status formats as
 transient errors

aria2 prefixes HTTP statuses with 'status=XXX' in its onDownloadError payload instead of 'HTTP XXX'. These were falling through the transient check and causing instant, permanent failure instead of triggering the escalating backoff schedule.
---
 src-tauri/src/retry.rs | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/src-tauri/src/retry.rs b/src-tauri/src/retry.rs
index b163a5d..5ccfbc3 100644
--- a/src-tauri/src/retry.rs
+++ b/src-tauri/src/retry.rs
@@ -102,7 +102,7 @@ pub fn is_transient_network_error(message: &str) -> bool {
 
     let m = message.to_ascii_lowercase();
 
-    const TRANSIENT: [&str; 23] = [
+    const TRANSIENT: [&str; 34] = [
         // reqwest / hyper / OS socket-layer
         "timed out",
         "timeout",
@@ -126,6 +126,18 @@ pub fn is_transient_network_error(message: &str) -> bool {
         "http 429",
         "http error 429",
         "429 too many requests",
+        // aria2c HTTP error formats
+        "status=408",
+        "status=429",
+        "status=500",
+        "status=502",
+        "status=503",
+        "status=504",
+        "status=520",
+        "status=521",
+        "status=522",
+        "status=523",
+        "status=524",
         // aria2c log phrasing
         "connection was closed",
         "timeout.",
@@ -251,6 +263,7 @@ mod tests {
         assert!(is_transient_network_error("HTTP 429 Too Many Requests"));
         assert!(is_transient_network_error("HTTP Error 429: Too Many Requests"));
         assert!(is_transient_network_error("429 too many requests"));
+        assert!(is_transient_network_error("The response status is not successful. status=429"));
     }
 
     #[test]
@@ -263,6 +276,8 @@ mod tests {
         ));
         assert!(is_transient_network_error("Timeout."));
         assert!(is_transient_network_error("network is unreachable"));
+        assert!(is_transient_network_error("The response status is not successful. status=503"));
+        assert!(is_transient_network_error("The response status is not successful. status=502"));
     }
 
     // --- transient classification: negative cases -------------------------