From 4560a8042c10e2a4a3ec94d134a2112913208503 Mon Sep 17 00:00:00 2001
From: Dionisio <dpozo@futurespace.es>
Date: Fri, 13 Feb 2026 22:44:20 +0100
Subject: [PATCH] fix(routes): serve pages with clean URLs (no .html extension)

Add server-side routes for /profile, /admin, and /shared that
serve their respective HTML pages directly (same pattern already
used for /login). Updated all frontend references to use clean
URLs instead of .html extensions.

Fixes #99
---
 src/interfaces/web/mod.rs | 20 +++++++++++++++++++-
 static/admin.html         |  2 +-
 static/js/app.js          | 12 ++++++------
 static/js/auth.js         |  6 +++---
 static/js/shared.js       |  2 +-
 static/profile.html       |  2 +-
 static/shared.html        |  6 +++---
 7 files changed, 34 insertions(+), 16 deletions(-)

diff --git a/src/interfaces/web/mod.rs b/src/interfaces/web/mod.rs
index 06ca0fa4..eed19218 100644
--- a/src/interfaces/web/mod.rs
+++ b/src/interfaces/web/mod.rs
@@ -14,8 +14,11 @@ pub fn create_web_routes() -> Router<AppState> {
     let static_path = config.static_path.clone();
 
     Router::new()
-        // Add specific route for login
+        // Add specific routes for clean URLs (without .html)
         .route("/login", get(serve_login_page))
+        .route("/profile", get(serve_profile_page))
+        .route("/admin", get(serve_admin_page))
+        .route("/shared", get(serve_shared_page))
         // Serve static files
         .fallback_service(
             ServeDir::new(static_path)
@@ -25,4 +28,19 @@ pub fn create_web_routes() -> Router<AppState> {
 /// Serve the login page
 async fn serve_login_page() -> Html<&'static str> {
     Html(include_str!("../../../static/login.html"))
+}
+
+/// Serve the profile page
+async fn serve_profile_page() -> Html<&'static str> {
+    Html(include_str!("../../../static/profile.html"))
+}
+
+/// Serve the admin page
+async fn serve_admin_page() -> Html<&'static str> {
+    Html(include_str!("../../../static/admin.html"))
+}
+
+/// Serve the shared page
+async fn serve_shared_page() -> Html<&'static str> {
+    Html(include_str!("../../../static/shared.html"))
 }
\ No newline at end of file
diff --git a/static/admin.html b/static/admin.html
index 3dbb17b7..7732e747 100644
--- a/static/admin.html
+++ b/static/admin.html
@@ -219,7 +219,7 @@ details[open] summary{margin-bottom:14px;color:#ff5e3a}
     <div class="access-icon"><i class="fas fa-lock"></i></div>
     <h2>Access Denied</h2>
     <p>Administrator privileges required to access this panel.</p>
-    <a href="/login.html"><i class="fas fa-sign-in-alt"></i> Sign in</a>
+    <a href="/login"><i class="fas fa-sign-in-alt"></i> Sign in</a>
   </div>
 
   <div id="main-content" style="display:none">
diff --git a/static/js/app.js b/static/js/app.js
index 5df43ba4..159cafb1 100644
--- a/static/js/app.js
+++ b/static/js/app.js
@@ -207,7 +207,7 @@ function setupUserMenu() {
     if (adminBtn) {
         adminBtn.addEventListener('click', () => {
             wrapper.classList.remove('open');
-            window.location.href = '/admin.html';
+            window.location.href = '/admin';
         });
     }
     
@@ -215,7 +215,7 @@ function setupUserMenu() {
     if (profileBtn) {
         profileBtn.addEventListener('click', () => {
             wrapper.classList.remove('open');
-            window.location.href = '/profile.html';
+            window.location.href = '/profile';
         });
     }
     
@@ -1627,7 +1627,7 @@ async function checkAuthentication() {
                 if (!exchangeResponse.ok) {
                     const errText = await exchangeResponse.text();
                     console.error('OIDC token exchange failed:', exchangeResponse.status, errText);
-                    window.location.href = '/login.html?source=oidc_error';
+                    window.location.href = '/login?source=oidc_error';
                     return;
                 }
                 
@@ -1677,7 +1677,7 @@ async function checkAuthentication() {
                 }
             } catch (err) {
                 console.error('OIDC exchange error:', err);
-                window.location.href = '/login.html?source=oidc_error';
+                window.location.href = '/login?source=oidc_error';
                 return;
             }
         }
@@ -1688,7 +1688,7 @@ async function checkAuthentication() {
         if (!token) {
             console.log('No token found, redirecting to login');
             // Avoid potential loop by adding a parameter
-            const redirectUrl = '/login.html?source=app';
+            const redirectUrl = '/login?source=app';
             window.location.href = redirectUrl;
             return;
         }
@@ -1931,7 +1931,7 @@ function logout() {
     sessionStorage.removeItem('redirect_count');
     
     // Redirect to login page with correct path
-    window.location.href = '/login.html';
+    window.location.href = '/login';
 }
 
 /**
diff --git a/static/js/auth.js b/static/js/auth.js
index 9ec6885b..a6d2157d 100644
--- a/static/js/auth.js
+++ b/static/js/auth.js
@@ -1189,14 +1189,14 @@ function redirectToMainApp() {
                 // Ultimate fallback - clear everything and go to a special error page
                 localStorage.clear();
                 sessionStorage.clear();
-                window.location.href = '/login.html?critical=redirect_error';
+                window.location.href = '/login?critical=redirect_error';
             }
         }, 50);
     } catch (error) {
         console.error('Fatal error in redirectToMainApp:', error);
         // Emergency fallback
         try {
-            window.location.href = '/login.html?error=redirect_fatal';
+            window.location.href = '/login?error=redirect_fatal';
         } catch (e) {
             // Nothing more we can do
             alert('Critical redirect error. Please reload the page and try again.');
@@ -1215,5 +1215,5 @@ function logout() {
     localStorage.removeItem(REFRESH_TOKEN_KEY);
     localStorage.removeItem(TOKEN_EXPIRY_KEY);
     localStorage.removeItem(USER_DATA_KEY);
-    window.location.href = '/login.html';
+    window.location.href = '/login';
 }
\ No newline at end of file
diff --git a/static/js/shared.js b/static/js/shared.js
index 0006dea7..00c06dd6 100644
--- a/static/js/shared.js
+++ b/static/js/shared.js
@@ -15,7 +15,7 @@ function checkAuthentication() {
     
     if (!token || !tokenExpiry || new Date(tokenExpiry) < new Date()) {
         // No token or expired token
-        window.location.href = '/login.html';
+        window.location.href = '/login';
         return;
     }
     
diff --git a/static/profile.html b/static/profile.html
index 478a8739..61586352 100644
--- a/static/profile.html
+++ b/static/profile.html
@@ -143,7 +143,7 @@ body{background:#f5f7fa;color:#1e293b;min-height:100vh;display:flex;flex-directi
     <div class="err-icon"><i class="fas fa-lock"></i></div>
     <h2>Not Authenticated</h2>
     <p>Please sign in to view your profile.</p>
-    <a href="/login.html"><i class="fas fa-sign-in-alt"></i> Sign in</a>
+    <a href="/login"><i class="fas fa-sign-in-alt"></i> Sign in</a>
   </div>
 
   <div id="main-content" style="display:none">
diff --git a/static/shared.html b/static/shared.html
index 85cd8b16..0eeb1afe 100644
--- a/static/shared.html
+++ b/static/shared.html
@@ -25,7 +25,7 @@
                 <i class="fas fa-folder"></i>
                 <span data-i18n="nav.files">Files</span>
             </a>
-            <a href="/shared.html" class="nav-item active">
+            <a href="/shared" class="nav-item active">
                 <i class="fas fa-share-alt"></i>
                 <span data-i18n="nav.shared">Shared</span>
             </a>
@@ -250,7 +250,7 @@
         (function() {
             const token = localStorage.getItem('oxicloud_token');
             if (!token) {
-                window.location.href = '/auth/login.html';
+                window.location.href = '/login';
                 return;
             }
             // Set user avatar initials
@@ -281,7 +281,7 @@
                 logoutBtn.addEventListener('click', () => {
                     localStorage.removeItem('oxicloud_token');
                     localStorage.removeItem('oxicloud_user');
-                    window.location.href = '/auth/login.html';
+                    window.location.href = '/login';
                 });
             }
         })();