Improve App initialization logic with better error handling and logging to prevent infinite loading state

backend/src/diff.rs

@@ -85,7 +85,7 @@ pub fn diff_torrents(old: &[Torrent], new: &[Torrent]) -> DiffResult {
             has_changes = true;
 
             // Log status changes for debugging
-            tracing::info!(
+            tracing::debug!(
                 "Torrent status changed: {} ({}) {:?} -> {:?}",
                 new_t.name, new_t.hash, old_t.status, new_t.status
             );

backend/src/handlers/auth.rs

@@ -1,4 +1,4 @@
-use crate::{db::Db, AppState};
+use crate::AppState;
 use axum::{
     extract::{State, Json},
     http::StatusCode,
@@ -15,6 +15,7 @@ pub struct LoginRequest {
     password: String,
 }
 
+#[allow(dead_code)]
 #[derive(Serialize, ToSchema)]
 pub struct UserResponse {
     username: String,
@@ -35,11 +36,16 @@ pub async fn login_handler(
     jar: CookieJar,
     Json(payload): Json<LoginRequest>,
 ) -> impl IntoResponse {
+    tracing::info!("Login attempt for user: {}", payload.username);
+
     let user = match state.db.get_user_by_username(&payload.username).await {
         Ok(Some(u)) => u,
-        Ok(None) => return (StatusCode::UNAUTHORIZED, "Invalid credentials").into_response(),
+        Ok(None) => {
+            tracing::warn!("Login failed: User not found for {}", payload.username);
+            return (StatusCode::UNAUTHORIZED, "Invalid credentials").into_response();
+        }
         Err(e) => {
-            tracing::error!("DB error during login: {}", e);
+            tracing::error!("DB error during login for {}: {}", payload.username, e);
             return (StatusCode::INTERNAL_SERVER_ERROR, "Database error").into_response();
         }
     };
@@ -48,6 +54,8 @@ pub async fn login_handler(
 
     match bcrypt::verify(&payload.password, &password_hash) {
         Ok(true) => {
+            tracing::info!("Password verified for user: {}", payload.username);
+
             // Create session
             let token: String = (0..32).map(|_| {
                 use rand::{distributions::Alphanumeric, Rng};
@@ -59,20 +67,28 @@ pub async fn login_handler(
             let expires_at = time::OffsetDateTime::now_utc().unix_timestamp() + expires_in;
 
             if let Err(e) = state.db.create_session(user_id, &token, expires_at).await {
-                tracing::error!("Failed to create session: {}", e);
+                tracing::error!("Failed to create session for {}: {}", payload.username, e);
                 return (StatusCode::INTERNAL_SERVER_ERROR, "Failed to create session").into_response();
             }
 
             let cookie = Cookie::build(("auth_token", token))
                 .path("/")
                 .http_only(true)
-                .same_site(SameSite::Strict)
+                .same_site(SameSite::Lax)
                 .max_age(Duration::seconds(expires_in))
                 .build();
 
+            tracing::info!("Session created and cookie set for user: {}", payload.username);
             (StatusCode::OK, jar.add(cookie), "Login successful").into_response()
         }
-        _ => (StatusCode::UNAUTHORIZED, "Invalid credentials").into_response(),
+        Ok(false) => {
+            tracing::warn!("Login failed: Invalid password for {}", payload.username);
+            (StatusCode::UNAUTHORIZED, "Invalid credentials").into_response()
+        }
+        Err(e) => {
+            tracing::error!("Bcrypt error for {}: {}", payload.username, e);
+            (StatusCode::INTERNAL_SERVER_ERROR, "Auth error").into_response()
+        }
     }
 }
 

frontend/src/app.rs

@@ -22,44 +22,64 @@ pub fn App() -> impl IntoView {
     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 {
-            // 1. Check Setup Status
-            let setup_res = gloo_net::http::Request::get("/api/setup/status").send().await;
-            if let Ok(resp) = setup_res {
-                if let Ok(status) = resp.json::<SetupStatus>().await {
-                    if !status.completed {
-                        // Redirect to setup if not completed
-                        let navigate = use_navigate();
-                        navigate("/setup", Default::default());
-                        set_is_loading.set(false);
-                        return;
-                    }
-                }
-            }
+        // Check Auth & Setup Status on load
+        create_effect(move |_| {
+            spawn_local(async move {
+                logging::log!("App initialization started...");
 
-            // 2. Check Auth Status
-            let auth_res = gloo_net::http::Request::get("/api/auth/check").send().await;
-            if let Ok(resp) = auth_res {
-                if resp.status() == 200 {
-                    set_is_authenticated.set(true);
+                // 1. Check Setup Status
+                logging::log!("Checking setup status...");
+                let setup_res = gloo_net::http::Request::get("/api/setup/status").send().await;
 
-                    // Initialize push notifications logic only if authenticated
-                    // ... (Push notification logic moved here or kept global but guarded)
-                } else {
-                    let navigate = use_navigate();
-                    // If we are already on login or setup, don't redirect loop
-                    let pathname = window().location().pathname().unwrap_or_default();
-                    if pathname != "/login" && pathname != "/setup" {
-                         navigate("/login", Default::default());
+                match setup_res {
+                    Ok(resp) => {
+                        if resp.ok() {
+                            match resp.json::<SetupStatus>().await {
+                                Ok(status) => {
+                                    logging::log!("Setup status: completed={}", status.completed);
+                                    if !status.completed {
+                                        logging::log!("Setup not completed, redirecting to /setup");
+                                        let navigate = use_navigate();
+                                        navigate("/setup", Default::default());
+                                        set_is_loading.set(false);
+                                        return;
+                                    }
+                                }
+                                Err(e) => logging::error!("Failed to parse setup status: {}", e),
+                            }
+                        } else {
+                            logging::error!("Setup status request failed: {}", resp.status());
+                        }
                     }
+                    Err(e) => logging::error!("Network error checking setup status: {}", e),
                 }
-            }
-            set_is_loading.set(false);
+
+                // 2. Check Auth Status
+                logging::log!("Checking auth status...");
+                let auth_res = gloo_net::http::Request::get("/api/auth/check").send().await;
+
+                match auth_res {
+                    Ok(resp) => {
+                        logging::log!("Auth check status: {}", resp.status());
+                        if resp.status() == 200 {
+                            logging::log!("Authenticated!");
+                            set_is_authenticated.set(true);
+                        } else {
+                            logging::log!("Not authenticated, checking if redirect needed");
+                            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),
+                }
+
+                logging::log!("App initialization finished, disabling loader.");
+                set_is_loading.set(false);
+            });
         });
-    });
-
     // Initialize push notifications after user grants permission (Only if authenticated)
     create_effect(move |_| {
         if is_authenticated.get() {

frontend/src/components/auth/login.rs

@@ -1,6 +1,6 @@
 use leptos::*;
 use leptos_router::*;
-use serde::{Deserialize, Serialize};
+use serde::Serialize;
 
 #[derive(Serialize)]
 struct LoginRequest {
@@ -20,6 +20,8 @@ pub fn Login() -> impl IntoView {
         set_loading.set(true);
         set_error.set(None);
 
+        logging::log!("Attempting login for user: {}", username.get());
+
         spawn_local(async move {
             let req = LoginRequest {
                 username: username.get(),
@@ -32,15 +34,19 @@ pub fn Login() -> impl IntoView {
 
             match client.send().await {
                 Ok(resp) => {
+                    logging::log!("Login response status: {}", resp.status());
                     if resp.ok() {
-                        // Redirect to home on success
-                        let navigate = use_navigate();
-                        navigate("/", Default::default());
+                        logging::log!("Login successful, redirecting...");
+                        // Force a full reload to re-run auth checks in App.rs
+                        let _ = window().location().set_href("/");
                     } 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(_) => {
+                Err(e) => {
+                    logging::error!("Network error: {}", e);
                     set_error.set(Some("Bağlantı hatası".to_string()));
                 }
             }

frontend/src/components/auth/setup.rs

@@ -1,6 +1,6 @@
 use leptos::*;
 use leptos_router::*;
-use serde::{Deserialize, Serialize};
+use serde::Serialize;
 
 #[derive(Serialize)]
 struct SetupRequest {
@@ -49,9 +49,8 @@ pub fn Setup() -> impl IntoView {
             match client.send().await {
                 Ok(resp) => {
                     if resp.ok() {
-                        // Redirect to login after setup
-                        let navigate = use_navigate();
-                        navigate("/login", Default::default());
+                        // Redirect to login after setup (full reload to be safe)
+                        let _ = window().location().set_href("/login");
                     } else {
                         let text = resp.text().await.unwrap_or_default();
                         set_error.set(Some(format!("Hata: {}", text)));