fix: redirect root url

think-in-universe · think-in-universe · commit e308df5ac952 · 2025-11-26T17:45:28.000+08:00
diff --git a/crates/api/src/routes/mod.rs b/crates/api/src/routes/mod.rs
@@ -4,7 +4,7 @@ pub mod attestation;
 pub mod oauth;
 pub mod users;
 
-use axum::{middleware::from_fn_with_state, routing::get, Json, Router};
+use axum::{middleware::from_fn_with_state, response::Redirect, routing::get, Json, Router};
 use serde::Serialize;
 use tower_http::cors::{Any, CorsLayer};
 use utoipa::ToSchema;
@@ -78,6 +78,8 @@ pub fn create_router_with_cors(app_state: AppState, allowed_origins: Vec<String>
 
     // Build the base router
     let router = Router::new()
+        // Redirect root to /?v=1 to workaround the cache issue
+        .route("/", get(|| async { Redirect::temporary("/?v=1") }))
         .route("/health", get(health_check))
         .nest("/v1/auth", auth_routes)
         .nest("/v1/users", user_routes)
diff --git a/crates/api/src/static_files.rs b/crates/api/src/static_files.rs
@@ -19,21 +19,17 @@ fn frontend_dir() -> PathBuf {
 /// Serve static files with SPA fallback and proper cache headers.
 pub async fn static_handler(req: Request<Body>) -> Response {
     let frontend_dir = frontend_dir();
-    let path = req.uri().path().to_owned();
-    let is_root = path == "/";
 
-    let has_extension = path
+    let has_extension = req
+        .uri()
+        .path()
         .rsplit('/')
         .next()
         .map(|segment| segment.contains('.'))
         .unwrap_or(false);
 
     // Serve static files if they have an extension, otherwise fallback to the index.html file
-    let result = if is_root {
-        ServeFile::new(frontend_dir.join("index.html"))
-            .oneshot(req)
-            .await
-    } else if has_extension {
+    let result = if has_extension {
         ServeDir::new(frontend_dir.clone()).oneshot(req).await
     } else {
         ServeDir::new(frontend_dir.clone())
@@ -50,7 +46,7 @@ pub async fn static_handler(req: Request<Body>) -> Response {
                     .map(|ct| ct.contains("text/html"))
                     .unwrap_or(false);
                 // Don't cache HTML files to allow for SPA updates
-                let cache_value = if is_html || is_root {
+                let cache_value = if is_html {
                     "no-cache, no-store, must-revalidate"
                 } else {
                     "public, max-age=31536000, immutable"
