fix: add timeout to SCGI requests to prevent background loop hang

- Create frontend/src/api/mod.rs with centralized HTTP client and error handling
- Implement api::auth module (login, logout, check_auth, get_user)
- Implement api::torrent module (add, action, delete, start, stop, set_label, set_priority)
- Implement api::setup module (get_status, setup)
- Implement api::settings module (set_global_limits)
- Implement api::push module (get_public_key, subscribe)
- Update all components to use api service layer instead of direct gloo_net calls
- Add thiserror dependency for error handling

Cargo.lock

@@ -1218,6 +1218,7 @@ dependencies = [
  "serde_json",
  "shared",
  "tailwind_fuse",
+ "thiserror 2.0.18",
  "uuid",
  "wasm-bindgen",
  "wasm-bindgen-futures",

backend/src/handlers/mod.rs

@@ -11,12 +11,10 @@ use axum::{
     BoxError,
 };
 use rust_embed::RustEmbed;
-use serde::Deserialize;
 use shared::{
-    GlobalLimitRequest, SetFilePriorityRequest, SetLabelRequest, TorrentActionRequest, TorrentFile,
+    AddTorrentRequest, GlobalLimitRequest, SetFilePriorityRequest, SetLabelRequest, TorrentActionRequest,
-    TorrentPeer, TorrentTracker,
+    TorrentFile,     TorrentPeer, TorrentTracker,
 };
-use utoipa::ToSchema;
 
 pub mod auth;
 pub mod setup;
@@ -25,13 +23,6 @@ pub mod setup;
 #[folder = "../frontend/dist"]
 pub struct Asset;
 
-#[derive(Deserialize, ToSchema)]
-pub struct AddTorrentRequest {
-    /// Magnet link or Torrent file URL
-    #[schema(example = "magnet:?xt=urn:btih:...")]
-    uri: String,
-}
-
 pub async fn static_handler(uri: Uri) -> impl IntoResponse {
     let mut path = uri.path().trim_start_matches('/').to_string();
 

backend/src/main.rs

@@ -45,6 +45,7 @@ pub struct AppState {
     pub db: db::Db,
     #[cfg(feature = "push-notifications")]
     pub push_store: push::PushSubscriptionStore,
+    pub notify_poll: Arc<tokio::sync::Notify>,
 }
 
 async fn auth_middleware(
@@ -125,7 +126,7 @@ struct Args {
     ),
     components(
         schemas(
-            handlers::AddTorrentRequest,
+            shared::AddTorrentRequest,
             shared::TorrentActionRequest,
             shared::Torrent,
             shared::TorrentStatus,
@@ -172,7 +173,7 @@ struct ApiDoc;
     ),
     components(
         schemas(
-            handlers::AddTorrentRequest,
+            shared::AddTorrentRequest,
             shared::TorrentActionRequest,
             shared::Torrent,
             shared::TorrentStatus,
@@ -336,6 +337,8 @@ async fn main() {
     #[cfg(not(feature = "push-notifications"))]
     let push_store = ();
 
+    let notify_poll = Arc::new(tokio::sync::Notify::new());
+
     let app_state = AppState {
         tx: tx.clone(),
         event_bus: event_bus.clone(),
@@ -343,6 +346,7 @@ async fn main() {
         db: db.clone(),
         #[cfg(feature = "push-notifications")]
         push_store,
+        notify_poll: notify_poll.clone(),
     };
 
     // Spawn background task to poll rTorrent
@@ -351,6 +355,7 @@ async fn main() {
     let socket_path = args.socket.clone(); // Clone for background task
     #[cfg(feature = "push-notifications")]
     let push_store_clone = app_state.push_store.clone();
+    let notify_poll_clone = notify_poll.clone();
 
     tokio::spawn(async move {
         let client = xmlrpc::RtorrentClient::new(&socket_path);
@@ -359,6 +364,14 @@ async fn main() {
         let mut backoff_duration = Duration::from_secs(1);
 
         loop {
+            // Determine polling interval based on active clients
+            let active_clients = event_bus_tx.receiver_count();
+            let loop_interval = if active_clients > 0 {
+                Duration::from_secs(1)
+            } else {
+                Duration::from_secs(30)
+            };
+
             // 1. Fetch Torrents
             let torrents_result = sse::fetch_torrents(&client).await;
 
@@ -429,6 +442,14 @@ async fn main() {
                     }
 
                     previous_torrents = new_torrents;
+
+                    // Success case: wait for the determined interval OR a wakeup notification
+                    tokio::select! {
+                        _ = tokio::time::sleep(loop_interval) => {},
+                        _ = notify_poll_clone.notified() => {
+                            tracing::debug!("Background loop awakened by new client connection");
+                        }
+                    }
                 }
                 Err(e) => {
                     tracing::error!("Error fetching torrents in background: {}", e);
@@ -449,20 +470,15 @@ async fn main() {
                         "Backoff: Sleeping for {:?} due to rTorrent error.",
                         backoff_duration
                     );
+                    
+                    tokio::time::sleep(backoff_duration).await;
                 }
             }
 
             // Handle Stats
-            match stats_result {
+            if let Ok(stats) = stats_result {
-                Ok(stats) => {
+                let _ = event_bus_tx.send(AppEvent::Stats(stats));
-                    let _ = event_bus_tx.send(AppEvent::Stats(stats));
-                }
-                Err(e) => {
-                    tracing::warn!("Error fetching global stats: {}", e);
-                }
             }
-
-            tokio::time::sleep(backoff_duration).await;
         }
     });
 

backend/src/scgi.rs

@@ -11,6 +11,8 @@ pub enum ScgiError {
     #[allow(dead_code)]
     #[error("Protocol Error: {0}")]
     Protocol(String),
+    #[error("Timeout: SCGI request took too long")]
+    Timeout,
 }
 
 pub struct ScgiRequest {
@@ -78,20 +80,30 @@ impl ScgiRequest {
 }
 
 pub async fn send_request(socket_path: &str, request: ScgiRequest) -> Result<Bytes, ScgiError> {
-    let mut stream = UnixStream::connect(socket_path).await?;
+    let perform_request = async {
-    let data = request.encode();
+        let mut stream = UnixStream::connect(socket_path).await?;
-    stream.write_all(&data).await?;
+        let data = request.encode();
+        stream.write_all(&data).await?;
 
-    let mut response = Vec::new();
+        let mut response = Vec::new();
-    stream.read_to_end(&mut response).await?;
+        stream.read_to_end(&mut response).await?;
+        Ok::<Vec<u8>, std::io::Error>(response)
+    };
+
+    let response = tokio::time::timeout(std::time::Duration::from_secs(10), perform_request)
+        .await
+        .map_err(|_| ScgiError::Timeout)??;
 
     let double_newline = b"\r\n\r\n";
-    if let Some(pos) = response
+    let mut response_vec = response;
+    if let Some(pos) = response_vec
         .windows(double_newline.len())
         .position(|window| window == double_newline)
     {
-        Ok(Bytes::from(response.split_off(pos + double_newline.len())))
+        Ok(Bytes::from(
+            response_vec.split_off(pos + double_newline.len()),
+        ))
     } else {
-        Ok(Bytes::from(response))
+        Ok(Bytes::from(response_vec))
     }
 }

backend/src/sse.rs

@@ -195,6 +195,9 @@ pub async fn fetch_global_stats(client: &RtorrentClient) -> Result<GlobalStats,
 pub async fn sse_handler(
     State(state): State<AppState>,
 ) -> Sse<impl Stream<Item = Result<Event, Infallible>>> {
+    // Notify background worker to wake up and poll immediately
+    state.notify_poll.notify_one();
+
     // Get initial value synchronously (from the watch channel's current state)
     let initial_rx = state.tx.subscribe();
     let initial_torrents = initial_rx.borrow().clone();

frontend/Cargo.toml

@@ -30,3 +30,4 @@ base64 = "0.22.1"
 serde-wasm-bindgen = "0.6.5"
 leptos-use = "0.13"
 codee = "0.2"
+thiserror = "2.0"

frontend/src/api/mod.rs

@@ -0,0 +1,243 @@
+use gloo_net::http::Request;
+use shared::{AddTorrentRequest, TorrentActionRequest};
+use thiserror::Error;
+
+#[derive(Debug, Error)]
+pub enum ApiError {
+    #[error("Network error")]
+    Network,
+    #[error("Server error: {status}")]
+    Server { status: u16 },
+    #[error("Login failed")]
+    LoginFailed,
+    #[error("Unauthorized")]
+    Unauthorized,
+    #[error("Too many requests")]
+    RateLimited,
+}
+
+fn base_url() -> String {
+    "/api".to_string()
+}
+
+pub mod auth {
+    use super::*;
+
+    #[derive(serde::Serialize)]
+    pub struct LoginRequest {
+        pub username: String,
+        pub password: String,
+        pub remember_me: bool,
+    }
+
+    pub async fn login(
+        username: &str,
+        password: &str,
+        remember_me: bool,
+    ) -> Result<(), ApiError> {
+        let req = LoginRequest {
+            username: username.to_string(),
+            password: password.to_string(),
+            remember_me,
+        };
+        let resp = Request::post(&format!("{}/auth/login", base_url()))
+            .json(&req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+
+        if resp.ok() {
+            Ok(())
+        } else if resp.status() == 429 {
+            Err(ApiError::RateLimited)
+        } else {
+            Err(ApiError::LoginFailed)
+        }
+    }
+
+    pub async fn logout() -> Result<(), ApiError> {
+        Request::post(&format!("{}/auth/logout", base_url()))
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+
+    pub async fn check_auth() -> Result<bool, ApiError> {
+        let resp = Request::get(&format!("{}/auth/check", base_url()))
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(resp.ok())
+    }
+
+    #[derive(serde::Deserialize)]
+    pub struct UserResponse {
+        pub username: String,
+    }
+
+    pub async fn get_user() -> Result<UserResponse, ApiError> {
+        let resp = Request::get(&format!("{}/auth/check", base_url()))
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        let user = resp.json().await.map_err(|_| ApiError::Network)?;
+        Ok(user)
+    }
+}
+
+pub mod setup {
+    use super::*;
+
+    #[derive(serde::Serialize)]
+    pub struct SetupRequest {
+        pub username: String,
+        pub password: String,
+    }
+
+    #[derive(serde::Deserialize)]
+    pub struct SetupStatusResponse {
+        pub completed: bool,
+    }
+
+    pub async fn get_status() -> Result<SetupStatusResponse, ApiError> {
+        let resp = Request::get(&format!("{}/setup/status", base_url()))
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        let status = resp.json().await.map_err(|_| ApiError::Network)?;
+        Ok(status)
+    }
+
+    pub async fn setup(username: &str, password: &str) -> Result<(), ApiError> {
+        let req = SetupRequest {
+            username: username.to_string(),
+            password: password.to_string(),
+        };
+        Request::post(&format!("{}/setup", base_url()))
+            .json(&req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+}
+
+pub mod settings {
+    use super::*;
+    use shared::GlobalLimitRequest;
+
+    pub async fn set_global_limits(req: &GlobalLimitRequest) -> Result<(), ApiError> {
+        Request::post(&format!("{}/settings/global-limits", base_url()))
+            .json(req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+}
+
+pub mod push {
+    use super::*;
+    use crate::store::PushSubscriptionData;
+
+    pub async fn get_public_key() -> Result<String, ApiError> {
+        let resp = Request::get(&format!("{}/push/public-key", base_url()))
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        let key = resp.text().await.map_err(|_| ApiError::Network)?;
+        Ok(key)
+    }
+
+    pub async fn subscribe(req: &PushSubscriptionData) -> Result<(), ApiError> {
+        Request::post(&format!("{}/push/subscribe", base_url()))
+            .json(req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+}
+
+pub mod torrent {
+    use super::*;
+
+    pub async fn add(uri: &str) -> Result<(), ApiError> {
+        let req = AddTorrentRequest {
+            uri: uri.to_string(),
+        };
+        Request::post(&format!("{}/torrents/add", base_url()))
+            .json(&req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+
+    pub async fn action(hash: &str, action: &str) -> Result<(), ApiError> {
+        let req = TorrentActionRequest {
+            hash: hash.to_string(),
+            action: action.to_string(),
+        };
+        Request::post(&format!("{}/torrents/action", base_url()))
+            .json(&req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+
+    pub async fn delete(hash: &str) -> Result<(), ApiError> {
+        action(hash, "delete").await
+    }
+
+    pub async fn delete_with_data(hash: &str) -> Result<(), ApiError> {
+        action(hash, "delete_with_data").await
+    }
+
+    pub async fn start(hash: &str) -> Result<(), ApiError> {
+        action(hash, "start").await
+    }
+
+    pub async fn stop(hash: &str) -> Result<(), ApiError> {
+        action(hash, "stop").await
+    }
+
+    pub async fn set_label(hash: &str, label: &str) -> Result<(), ApiError> {
+        use shared::SetLabelRequest;
+        let req = SetLabelRequest {
+            hash: hash.to_string(),
+            label: label.to_string(),
+        };
+        Request::post(&format!("{}/torrents/set_label", base_url()))
+            .json(&req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+
+    pub async fn set_priority(hash: &str, file_index: u32, priority: u8) -> Result<(), ApiError> {
+        use shared::SetFilePriorityRequest;
+        let req = SetFilePriorityRequest {
+            hash: hash.to_string(),
+            file_index,
+            priority,
+        };
+        Request::post(&format!("{}/torrents/set_priority", base_url()))
+            .json(&req)
+            .map_err(|_| ApiError::Network)?
+            .send()
+            .await
+            .map_err(|_| ApiError::Network)?;
+        Ok(())
+    }
+}

frontend/src/app.rs

@@ -3,100 +3,80 @@ use crate::components::toast::ToastContainer;
 use crate::components::torrent::table::TorrentTable;
 use crate::components::auth::login::Login;
 use crate::components::auth::setup::Setup;
+use crate::api;
 use leptos::*;
 use leptos_router::*;
-use serde::Deserialize;
-
-#[derive(Deserialize)]
-struct SetupStatus {
-    completed: bool,
-}
-
-#[derive(Deserialize)]
-struct UserResponse {
-    username: String,
-}
 
 #[component]
 pub fn App() -> impl IntoView {
     crate::store::provide_torrent_store();
 
-    // Auth State
     let (is_loading, set_is_loading) = create_signal(true);
     let (is_authenticated, set_is_authenticated) = create_signal(false);
 
-    // Check Auth & Setup Status on load
     create_effect(move |_| {
         spawn_local(async move {
             logging::log!("App initialization started...");
 
-            // 1. Check Setup Status
+            let setup_res = api::setup::get_status().await;
-            let setup_res = gloo_net::http::Request::get("/api/setup/status").send().await;
 
             match setup_res {
-                Ok(resp) => {
+                Ok(status) => {
-                    if resp.ok() {
+                    if !status.completed {
-                        match resp.json::<SetupStatus>().await {
+                        logging::log!("Setup not completed, redirecting to /setup");
-                            Ok(status) => {
+                        let navigate = use_navigate();
-                                if !status.completed {
+                        navigate("/setup", Default::default());
-                                    logging::log!("Setup not completed, redirecting to /setup");
+                        set_is_loading.set(false);
-                                    let navigate = use_navigate();
+                        return;
-                                    navigate("/setup", Default::default());
-                                    set_is_loading.set(false);
-                                    return;
-                                }
-                            }
-                            Err(e) => logging::error!("Failed to parse setup status: {}", e),
-                        }
                     }
                 }
-                Err(e) => logging::error!("Network error checking setup status: {}", e),
+                Err(e) => logging::error!("Failed to get setup status: {:?}", e),
             }
 
-            // 2. Check Auth Status
+            let auth_res = api::auth::check_auth().await;
-            let auth_res = gloo_net::http::Request::get("/api/auth/check").send().await;
 
-                        match auth_res {
+            match auth_res {
-                            Ok(resp) => {
+                Ok(true) => {
-                                if resp.status() == 200 {
+                    logging::log!("Authenticated!");
-                                    logging::log!("Authenticated!");
 
-                                    // Parse user info
+                    if let Ok(user_info) = api::auth::get_user().await {
-                                    if let Ok(user_info) = resp.json::<UserResponse>().await {
+                        if let Some(store) = use_context::<crate::store::TorrentStore>() {
-                                        if let Some(store) = use_context::<crate::store::TorrentStore>() {
+                            store.user.set(Some(user_info.username));
-                                            store.user.set(Some(user_info.username));
-                                        }
-                                    }
-
-                                    set_is_authenticated.set(true);
-
-                                    // If user is already authenticated but on login/setup page, redirect to home
-                                    let pathname = window().location().pathname().unwrap_or_default();
-                                    if pathname == "/login" || pathname == "/setup" {
-                                        logging::log!("Already authenticated, redirecting to home");
-                                        let navigate = use_navigate();
-                                        navigate("/", Default::default());
-                                    }
-                                } else {
-                                    logging::log!("Not authenticated, redirecting to /login");
-                                    let navigate = use_navigate();
-                                    let pathname = window().location().pathname().unwrap_or_default();
-                                    if pathname != "/login" && pathname != "/setup" {
-                                         navigate("/login", Default::default());
-                                    }
-                                }
-                            }
-                            Err(e) => logging::error!("Network error checking auth status: {}", e),
                         }
+                    }
+
+                    set_is_authenticated.set(true);
+
+                    let pathname = window().location().pathname().unwrap_or_default();
+                    if pathname == "/login" || pathname == "/setup" {
+                        logging::log!("Already authenticated, redirecting to home");
+                        let navigate = use_navigate();
+                        navigate("/", Default::default());
+                    }
+                }
+                Ok(false) => {
+                    logging::log!("Not authenticated");
+
+                    let pathname = window().location().pathname().unwrap_or_default();
+                    if pathname != "/login" && pathname != "/setup" {
+                        let navigate = use_navigate();
+                        navigate("/login", Default::default());
+                    }
+                }
+                Err(e) => {
+                    logging::error!("Auth check failed: {:?}", e);
+                    let navigate = use_navigate();
+                    navigate("/login", Default::default());
+                }
+            }
+
             set_is_loading.set(false);
         });
     });
-    // Initialize push notifications (Only if authenticated)
+
     create_effect(move |_| {
         if is_authenticated.get() {
             spawn_local(async {
-                // ... (Push notification logic kept same, shortened for brevity in this replace)
-                // Wait a bit for service worker to be ready
                 gloo_timers::future::TimeoutFuture::new(2000).await;
 
                 if crate::utils::platform::supports_push_notifications() && !crate::utils::platform::is_safari() {

frontend/src/components/auth/login.rs

@@ -1,12 +1,5 @@
 use leptos::*;
-use serde::Serialize;
+use crate::api;
-
-#[derive(Serialize)]
-struct LoginRequest {
-    username: String,
-    password: String,
-    remember_me: bool,
-}
 
 #[component]
 pub fn Login() -> impl IntoView {
@@ -23,35 +16,31 @@ pub fn Login() -> impl IntoView {
 
         logging::log!("Attempting login for user: {}", username.get());
 
+        let username = username.get();
+        let password = password.get();
+        let remember_me = remember_me.get();
+
         spawn_local(async move {
-            let req = LoginRequest {
+            match api::auth::login(&username, &password, remember_me).await {
-                username: username.get(),
+                Ok(_) => {
-                password: password.get(),
+                    logging::log!("Login successful, redirecting...");
-                remember_me: remember_me.get(),
+                    let _ = window().location().set_href("/");
-            };
-
-            let client = gloo_net::http::Request::post("/api/auth/login")
-                .json(&req)
-                .expect("Failed to create request");
-
-            match client.send().await {
-                Ok(resp) => {
-                    logging::log!("Login response status: {}", resp.status());
-                    if resp.ok() {
-                        logging::log!("Login successful, redirecting...");
-                        // Force a full reload to re-run auth checks in App.rs
-                        let _ = window().location().set_href("/");
-                    } else if resp.status() == 429 {
-                        set_error.set(Some("Çok fazla başarısız deneme yaptınız. Lütfen bir süre bekleyip tekrar deneyin.".to_string()));
-                    } else {
-                        let text = resp.text().await.unwrap_or_default();
-                        logging::error!("Login failed: {}", text);
-                        set_error.set(Some("Kullanıcı adı veya şifre hatalı".to_string()));
-                    }
                 }
                 Err(e) => {
-                    logging::error!("Network error: {}", e);
+                    logging::error!("Login failed: {:?}", e);
-                    set_error.set(Some("Bağlantı hatası".to_string()));
+                    let msg = match e {
+                        crate::api::ApiError::RateLimited => {
+                            "Çok fazla başarısız deneme yaptınız. Lütfen bir süre bekleyip tekrar deneyin.".to_string()
+                        }
+                        crate::api::ApiError::Unauthorized | crate::api::ApiError::LoginFailed => {
+                            "Kullanıcı adı veya şifre hatalı".to_string()
+                        }
+                        crate::api::ApiError::Network => {
+                            "Bağlantı hatası".to_string()
+                        }
+                        _ => "Bir hata oluştu".to_string()
+                    };
+                    set_error.set(Some(msg));
                 }
             }
             set_loading.set(false);

frontend/src/components/auth/setup.rs

@@ -1,11 +1,5 @@
 use leptos::*;
-use serde::Serialize;
+use crate::api;
-
-#[derive(Serialize)]
-struct SetupRequest {
-    username: String,
-    password: String,
-}
 
 #[component]
 pub fn Setup() -> impl IntoView {
@@ -35,29 +29,18 @@ pub fn Setup() -> impl IntoView {
             return;
         }
 
+        let username = username.get();
+        let password = pass;
+
         spawn_local(async move {
-            let req = SetupRequest {
+            match api::setup::setup(&username, &password).await {
-                username: username.get(),
+                Ok(_) => {
-                password: pass,
+                    logging::log!("Setup completed successfully, redirecting...");
-            };
+                    let _ = window().location().set_href("/");
-
-            let client = gloo_net::http::Request::post("/api/setup")
-                .json(&req)
-                .expect("Failed to create request");
-
-            match client.send().await {
-                Ok(resp) => {
-                    if resp.ok() {
-                        // Redirect to home after setup (auto-login handled by backend)
-                        // Full reload to ensure auth state is refreshed
-                        let _ = window().location().set_href("/");
-                    } else {
-                        let text = resp.text().await.unwrap_or_default();
-                        set_error.set(Some(format!("Hata: {}", text)));
-                    }
                 }
-                Err(_) => {
+                Err(e) => {
-                    set_error.set(Some("Bağlantı hatası".to_string()));
+                    logging::error!("Setup failed: {:?}", e);
+                    set_error.set(Some("Kurulum başarısız oldu".to_string()));
                 }
             }
             set_loading.set(false);

frontend/src/components/layout/sidebar.rs

@@ -1,56 +1,52 @@
 use leptos::wasm_bindgen::JsCast;
 use leptos::*;
+use crate::api;
 
 #[component]
 pub fn Sidebar() -> impl IntoView {
     let store = use_context::<crate::store::TorrentStore>().expect("store not provided");
 
-    let total_count = move || store.torrents.get().len();
+    let total_count = move || store.torrents.with(|map| map.len());
     let downloading_count = move || {
-        store
+        store.torrents.with(|map| {
-            .torrents
+            map.values()
-            .get()
+                .filter(|t| t.status == shared::TorrentStatus::Downloading)
-            .iter()
+                .count()
-            .filter(|t| t.status == shared::TorrentStatus::Downloading)
+        })
-            .count()
     };
     let seeding_count = move || {
-        store
+        store.torrents.with(|map| {
-            .torrents
+            map.values()
-            .get()
+                .filter(|t| t.status == shared::TorrentStatus::Seeding)
-            .iter()
+                .count()
-            .filter(|t| t.status == shared::TorrentStatus::Seeding)
+        })
-            .count()
     };
     let completed_count = move || {
-        store
+        store.torrents.with(|map| {
-            .torrents
+            map.values()
-            .get()
+                .filter(|t| {
-            .iter()
+                    t.status == shared::TorrentStatus::Seeding
-            .filter(|t| {
+                        || (t.status == shared::TorrentStatus::Paused && t.percent_complete >= 100.0)
-                t.status == shared::TorrentStatus::Seeding
+                })
-                    || (t.status == shared::TorrentStatus::Paused && t.percent_complete >= 100.0)
+                .count()
-            })
+        })
-            .count()
     };
     let paused_count = move || {
-        store
+        store.torrents.with(|map| {
-            .torrents
+            map.values()
-            .get()
+                .filter(|t| t.status == shared::TorrentStatus::Paused)
-            .iter()
+                .count()
-            .filter(|t| t.status == shared::TorrentStatus::Paused)
+        })
-            .count()
     };
     let inactive_count = move || {
-        store
+        store.torrents.with(|map| {
-            .torrents
+            map.values()
-            .get()
+                .filter(|t| {
-            .iter()
+                    t.status == shared::TorrentStatus::Paused
-            .filter(|t| {
+                        || t.status == shared::TorrentStatus::Error
-                t.status == shared::TorrentStatus::Paused
+                })
-                    || t.status == shared::TorrentStatus::Error
+                .count()
-            })
+        })
-            .count()
     };
 
     let close_drawer = move || {
@@ -76,12 +72,8 @@ pub fn Sidebar() -> impl IntoView {
 
     let handle_logout = move |_| {
         spawn_local(async move {
-            let client = gloo_net::http::Request::post("/api/auth/logout");
+            if api::auth::logout().await.is_ok() {
-            if let Ok(resp) = client.send().await {
+                let _ = window().location().set_href("/login");
-                if resp.ok() {
-                    // Force full reload to clear state
-                    let _ = window().location().set_href("/login");
-                }
             }
         });
     };

frontend/src/components/layout/statusbar.rs

@@ -2,6 +2,7 @@ use leptos::*;
 use leptos_use::storage::use_local_storage;
 use codee::string::FromToStringCodec;
 use shared::GlobalLimitRequest;
+use crate::api;
 
 fn format_bytes(bytes: i64) -> String {
     const UNITS: [&str; 6] = ["B", "KB", "MB", "GB", "TB", "PB"];
@@ -60,38 +61,23 @@ pub fn StatusBar() -> impl IntoView {
         let limit_type = limit_type.to_string();
         logging::log!("Setting {} limit to {}", limit_type, val);
 
+        let req = if limit_type == "down" {
+            GlobalLimitRequest {
+                max_download_rate: Some(val),
+                max_upload_rate: None,
+            }
+        } else {
+            GlobalLimitRequest {
+                max_download_rate: None,
+                max_upload_rate: Some(val),
+            }
+        };
+
         spawn_local(async move {
-            let req_body = if limit_type == "down" {
+            if let Err(e) = api::settings::set_global_limits(&req).await {
-                GlobalLimitRequest {
+                logging::error!("Failed to set limit: {:?}", e);
-                    max_download_rate: Some(val),
-                    max_upload_rate: None,
-                }
             } else {
-                GlobalLimitRequest {
+                logging::log!("Limit set successfully");
-                    max_download_rate: None,
-                    max_upload_rate: Some(val),
-                }
-            };
-
-            let client =
-                gloo_net::http::Request::post("/api/settings/global-limits").json(&req_body);
-
-            match client {
-                Ok(req) => match req.send().await {
-                    Ok(resp) => {
-                        if !resp.ok() {
-                            logging::error!(
-                                "Failed to set limit: {} {}",
-                                resp.status(),
-                                resp.status_text()
-                            );
-                        } else {
-                            logging::log!("Limit set successfully");
-                        }
-                    }
-                    Err(e) => logging::error!("Network error setting limit: {}", e),
-                },
-                Err(e) => logging::error!("Failed to create request: {}", e),
             }
         });
     };

frontend/src/components/torrent/add_torrent.rs

@@ -1,6 +1,7 @@
 use leptos::*;
 use leptos::html::Dialog;
 use crate::store::{show_toast_with_signal, TorrentStore};
+use crate::api;
 use shared::NotificationLevel;
 
 
@@ -17,7 +18,6 @@ pub fn AddTorrentModal(
     let (is_loading, set_loading) = create_signal(false);
     let (error_msg, set_error_msg) = create_signal(Option::<String>::None);
 
-    // Effect to open the dialog when the component mounts/renders
     create_effect(move |_| {
         if let Some(dialog) = dialog_ref.get() {
             let _ = dialog.show_modal();
@@ -35,46 +35,22 @@ pub fn AddTorrentModal(
         set_loading.set(true);
         set_error_msg.set(None);
 
+        let uri_val = uri_val;
         spawn_local(async move {
-            let req_body = serde_json::json!({
+            match api::torrent::add(&uri_val).await {
-                "uri": uri_val
+                Ok(_) => {
-            });
+                    logging::log!("Torrent added successfully");
-
+                    show_toast_with_signal(notifications, NotificationLevel::Success, "Torrent eklendi");
-            match gloo_net::http::Request::post("/api/torrents/add")
+                    set_loading.set(false);
-                .json(&req_body)
+                    if let Some(dialog) = dialog_ref.get() {
-            {
+                        dialog.close();
-                Ok(req) => {
-                    match req.send().await {
-                        Ok(resp) => {
-                            if resp.ok() {
-                                logging::log!("Torrent added successfully");
-                                show_toast_with_signal(notifications, NotificationLevel::Success, "Torrent eklendi");
-                                set_loading.set(false);
-                                if let Some(dialog) = dialog_ref.get() {
-                                    dialog.close();
-                                }
-                                on_close.call(());
-                            } else {
-                                let status = resp.status();
-                                let text = resp.text().await.unwrap_or_default();
-                                logging::error!("Failed to add torrent: {} - {}", status, text);
-                                show_toast_with_signal(notifications, NotificationLevel::Error, "Torrent eklenemedi");
-                                set_error_msg.set(Some(format!("Error {}: {}", status, text)));
-                                set_loading.set(false);
-                            }
-                        }
-                        Err(e) => {
-                            logging::error!("Network error: {}", e);
-                            show_toast_with_signal(notifications, NotificationLevel::Error, "Bağlantı hatası");
-                            set_error_msg.set(Some(format!("Network Error: {}", e)));
-                            set_loading.set(false);
-                        }
                     }
+                    on_close.call(());
                 }
                 Err(e) => {
-                    logging::error!("Serialization error: {}", e);
+                    logging::error!("Failed to add torrent: {:?}", e);
-                    show_toast_with_signal(notifications, NotificationLevel::Error, "İstek hatası");
+                    show_toast_with_signal(notifications, NotificationLevel::Error, "Torrent eklenemedi");
-                    set_error_msg.set(Some(format!("Request Error: {}", e)));
+                    set_error_msg.set(Some(format!("Error: {:?}", e)));
                     set_loading.set(false);
                 }
             }

frontend/src/components/torrent/table.rs

@@ -1,6 +1,7 @@
 use leptos::*;
 use leptos_use::{on_click_outside, use_timeout_fn};
 use crate::store::{get_action_messages, show_toast_with_signal};
+use crate::api;
 use shared::NotificationLevel;
 
 fn format_bytes(bytes: i64) -> String {
@@ -81,9 +82,10 @@ pub fn TorrentTable() -> impl IntoView {
     let sort_dir = create_rw_signal(SortDirection::Descending);
 
     let filtered_torrents = move || {
-        let mut torrents = store
+        // Convert HashMap values to Vec for filtering and sorting
-            .torrents
+        let torrents: Vec<shared::Torrent> = store.torrents.with(|map| map.values().cloned().collect());
-            .get()
+
+        let mut torrents = torrents
             .into_iter()
             .filter(|t| {
                 let filter = store.filter.get();
@@ -201,54 +203,33 @@ pub fn TorrentTable() -> impl IntoView {
 
     let on_action = move |(action, hash): (String, String)| {
         logging::log!("TorrentTable Action: {} on {}", action, hash);
-        // Note: Don't close menu here - ContextMenu's on_close handles it
-        // Closing here would dispose ContextMenu while still in callback chain
 
-        // Get action messages for toast (Clean Code: DRY)
         let (success_msg, error_msg) = get_action_messages(&action);
         let success_msg = success_msg.to_string();
         let error_msg = error_msg.to_string();
 
-        // Capture notifications signal before async (use_context unavailable in spawn_local)
         let notifications = store.notifications;
 
+        let hash = hash.clone();
+        let action = action.clone();
+
         spawn_local(async move {
-            let action_req = if action == "delete_with_data" {
+            let result = match action.as_str() {
-                "delete_with_data"
+                "delete" => api::torrent::delete(&hash).await,
-            } else {
+                "delete_with_data" => api::torrent::delete_with_data(&hash).await,
-                &action
+                "start" => api::torrent::start(&hash).await,
+                "stop" => api::torrent::stop(&hash).await,
+                _ => api::torrent::action(&hash, &action).await,
             };
 
-            let req_body = shared::TorrentActionRequest {
+            match result {
-                hash: hash.clone(),
+                Ok(_) => {
-                action: action_req.to_string(),
+                    logging::log!("Action {} executed successfully", action);
-            };
+                    show_toast_with_signal(notifications, NotificationLevel::Success, success_msg);
-
+                }
-            let client = gloo_net::http::Request::post("/api/torrents/action").json(&req_body);
-
-            match client {
-                Ok(req) => match req.send().await {
-                    Ok(resp) => {
-                        if !resp.ok() {
-                            logging::error!(
-                                "Failed to execute action: {} {}",
-                                resp.status(),
-                                resp.status_text()
-                            );
-                            show_toast_with_signal(notifications, NotificationLevel::Error, error_msg);
-                        } else {
-                            logging::log!("Action {} executed successfully", action);
-                            show_toast_with_signal(notifications, NotificationLevel::Success, success_msg);
-                        }
-                    }
-                    Err(e) => {
-                        logging::error!("Network error executing action: {}", e);
-                        show_toast_with_signal(notifications, NotificationLevel::Error, format!("{}: Bağlantı hatası", error_msg));
-                    }
-                },
                 Err(e) => {
-                    logging::error!("Failed to serialize request: {}", e);
+                    logging::error!("Action failed: {:?}", e);
-                    show_toast_with_signal(notifications, NotificationLevel::Error, error_msg);
+                    show_toast_with_signal(notifications, NotificationLevel::Error, format!("{}: {:?}", error_msg, e));
                 }
             }
         });
@@ -273,7 +254,7 @@ pub fn TorrentTable() -> impl IntoView {
                                  <div class="flex items-center">"Status" {move || sort_arrow(SortColumn::Status)}</div>
                             </th>
                             <th class="w-24 cursor-pointer hover:bg-base-300 group select-none" on:click=move |_| handle_sort(SortColumn::DownSpeed)>
-                                 <div class="flex items-center">"Down Speed" {move || sort_arrow(SortColumn::DownSpeed)}</div>
+                                 <div class="flex items-center">"DL Speed" {move || sort_arrow(SortColumn::DownSpeed)}</div>
                             </th>
                             <th class="w-24 cursor-pointer hover:bg-base-300 group select-none" on:click=move |_| handle_sort(SortColumn::UpSpeed)>
                                  <div class="flex items-center">"Up Speed" {move || sort_arrow(SortColumn::UpSpeed)}</div>
@@ -364,7 +345,7 @@ pub fn TorrentTable() -> impl IntoView {
                                                   (SortColumn::Size, "Size"),
                                                   (SortColumn::Progress, "Progress"),
                                                   (SortColumn::Status, "Status"),
-                                                  (SortColumn::DownSpeed, "Down Speed"),
+                                                  (SortColumn::DownSpeed, "DL Speed"),
                                                   (SortColumn::UpSpeed, "Up Speed"),
                                                   (SortColumn::ETA, "ETA"),
                                                   (SortColumn::AddedDate, "Date"),

frontend/src/lib.rs

@@ -1,8 +1,8 @@
 mod app;
-// mod models; // Removed
 mod components;
 pub mod utils;
 pub mod store;
+pub mod api;
 
 use leptos::*;
 use wasm_bindgen::prelude::*;

frontend/src/store.rs

@@ -113,9 +113,11 @@ impl FilterStatus {
     }
 }
 
+use std::collections::HashMap;
+
 #[derive(Clone, Copy, Debug)]
 pub struct TorrentStore {
-    pub torrents: RwSignal<Vec<Torrent>>,
+    pub torrents: RwSignal<HashMap<String, Torrent>>,
     pub filter: RwSignal<FilterStatus>,
     pub search_query: RwSignal<String>,
     pub global_stats: RwSignal<GlobalStats>,
@@ -124,7 +126,7 @@ pub struct TorrentStore {
 }
 
 pub fn provide_torrent_store() {
-    let torrents = create_rw_signal(Vec::<Torrent>::new());
+    let torrents = create_rw_signal(HashMap::new());
     let filter = create_rw_signal(FilterStatus::All);
     let search_query = create_rw_signal(String::new());
     let global_stats = create_rw_signal(GlobalStats::default());
@@ -193,12 +195,15 @@ pub fn provide_torrent_store() {
                                         if let Ok(event) = serde_json::from_str::<AppEvent>(&data_str) {
                                             match event {
                                                 AppEvent::FullList { torrents: list, .. } => {
-                                                    torrents.set(list);
+                                                    let map: HashMap<String, Torrent> = list
+                                                        .into_iter()
+                                                        .map(|t| (t.hash.clone(), t))
+                                                        .collect();
+                                                    torrents.set(map);
                                                 }
                                                 AppEvent::Update(update) => {
-                                                    torrents.update(|list| {
+                                                    torrents.update(|map| {
-                                                        if let Some(t) = list.iter_mut().find(|t| t.hash == update.hash)
+                                                        if let Some(t) = map.get_mut(&update.hash) {
-                                                        {
                                                             if let Some(name) = update.name {
                                                                 t.name = name;
                                                             }
@@ -314,21 +319,21 @@ use serde::{Deserialize, Serialize};
 use wasm_bindgen::prelude::*;
 
 #[derive(Serialize, Deserialize, Debug)]
-struct PushSubscriptionData {
+pub struct PushSubscriptionData {
-    endpoint: String,
+    pub endpoint: String,
-    keys: PushKeys,
+    pub keys: PushKeys,
 }
 
 #[derive(Serialize, Deserialize, Debug)]
-struct PushKeys {
+pub struct PushKeys {
-    p256dh: String,
+    pub p256dh: String,
-    auth: String,
+    pub auth: String,
 }
 
 /// Subscribe user to push notifications
 /// Requests notification permission if needed, then subscribes to push
 pub async fn subscribe_to_push_notifications() {
-    use gloo_net::http::Request;
+    use crate::api;
 
     // First, request notification permission if not already granted
     let window = web_sys::window().expect("window should exist");
@@ -347,36 +352,19 @@ pub async fn subscribe_to_push_notifications() {
 
     log::info!("Notification permission granted! Proceeding with push subscription...");
 
-
     // Get VAPID public key from backend
-    let public_key_response = match Request::get("/api/push/public-key").send().await {
+    let public_key = match api::push::get_public_key().await {
-        Ok(resp) => resp,
+        Ok(key) => key,
         Err(e) => {
             log::error!("Failed to get VAPID public key: {:?}", e);
             return;
         }
     };
 
-    let public_key_data: serde_json::Value = match public_key_response.json().await {
-        Ok(data) => data,
-        Err(e) => {
-            log::error!("Failed to parse VAPID public key: {:?}", e);
-            return;
-        }
-    };
-
-    let public_key = match public_key_data.get("publicKey").and_then(|v| v.as_str()) {
-        Some(key) => key,
-        None => {
-            log::error!("Missing publicKey in response");
-            return;
-        }
-    };
-
     log::info!("VAPID public key from backend: {} (len: {})", public_key, public_key.len());
 
     // Convert VAPID public key to Uint8Array
-    let public_key_array = match url_base64_to_uint8array(public_key) {
+    let public_key_array = match url_base64_to_uint8array(&public_key) {
         Ok(arr) => {
             log::info!("VAPID key converted to Uint8Array (length: {})", arr.length());
             arr
@@ -495,23 +483,8 @@ pub async fn subscribe_to_push_notifications() {
     };
 
     // Send to backend (subscription_data is already the struct we need)
-    let response = match Request::post("/api/push/subscribe")
+    if let Err(e) = api::push::subscribe(&subscription_data).await {
-        .json(&subscription_data)
+        log::error!("Failed to send subscription to backend: {:?}", e);
-        .expect("serialization should succeed")
-        .send()
-        .await
-    {
-        Ok(resp) => resp,
-        Err(e) => {
-            log::error!("Failed to send subscription to backend: {:?}", e);
-            return;
-        }
-    };
-
-    if response.ok() {
-        log::info!("Successfully subscribed to push notifications");
-    } else {
-        log::error!("Backend rejected push subscription: {:?}", response.status());
     }
 }
 

shared/src/lib.rs

@@ -130,3 +130,9 @@ pub struct SetLabelRequest {
     pub hash: String,
     pub label: String,
 }
+
+#[derive(Debug, Serialize, Deserialize, Clone, ToSchema)]
+pub struct AddTorrentRequest {
+    #[schema(example = "magnet:?xt=urn:btih:...")]
+    pub uri: String,
+}