refactor: Improve upload chunk reading error handling and remove unused imports.

Author: zerox80

backend/src/handlers/auth.rs

@@ -30,7 +30,7 @@ use axum::{
 };
 use chrono::{DateTime, Duration as ChronoDuration, Utc};
 use sha2::{Digest, Sha256};
-use sqlx::{self, FromRow};
+use sqlx;
 use std::{env, sync::OnceLock, time::Duration};
 
 /// Global salt for hashing login attempt identifiers.

backend/src/handlers/upload.rs

@@ -3,7 +3,7 @@ use crate::{
     models::{ErrorResponse, UploadResponse},
 };
 use axum::{
-    extract::{Multipart, State},
+    extract::Multipart,
     http::StatusCode,
     Json,
 };
@@ -159,16 +159,26 @@ pub async fn upload_image(
 
             let mut total_size = first_chunk.len();
 
-            while let Some(chunk) = field.chunk().await.map_err(|err| {
-                tracing::error!("Failed to read chunk: {}", err);
-                let _ = tokio::fs::remove_file(&filepath).await;
-                (
-                    StatusCode::INTERNAL_SERVER_ERROR,
-                    Json(ErrorResponse {
-                        error: format!("Failed to read file: {}", err),
-                    }),
-                )
-            })? {
+            loop {
+                let chunk_option = match field.chunk().await {
+                    Ok(opt) => opt,
+                    Err(err) => {
+                        tracing::error!("Failed to read chunk: {}", err);
+                        let _ = tokio::fs::remove_file(&filepath).await;
+                         return Err((
+                            StatusCode::INTERNAL_SERVER_ERROR,
+                            Json(ErrorResponse {
+                                error: format!("Failed to read file: {}", err),
+                            }),
+                        ));
+                    }
+                };
+
+                let chunk = match chunk_option {
+                    Some(c) => c,
+                    None => break,
+                };
+
                 total_size += chunk.len();
                 if total_size > MAX_FILE_SIZE {
                     let _ = tokio::fs::remove_file(&filepath).await;

backend/src/security/auth.rs

@@ -26,7 +26,7 @@ use axum::{
         request::Parts,
         HeaderMap, HeaderValue, StatusCode,
     },
-    Json,
+
 };
 use axum_extra::extract::cookie::{Cookie, CookieJar, SameSite};
 use chrono::{Duration, Utc};
@@ -38,7 +38,7 @@ use std::sync::LazyLock;
 use std::sync::OnceLock;
 use time::{Duration as TimeDuration, OffsetDateTime};
 
-use crate::db::{self, DbPool};
+use crate::db::DbPool;
 
 /// Global storage for the JWT secret key.
 /// Initialized once at application startup via init_jwt_secret().