refactor: ログを auth/error カテゴリに統一、auth_check 成功ログ削除

app.py

@@ -102,8 +102,8 @@ class RequestLoggingMiddleware(BaseHTTPMiddleware):
             # print(f"[RESPONSE] method={method} path={path} status={response.status_code} duration={duration:.3f}s{user_tag}")
             if response.status_code >= 400:
                 log_event(
-                    "response_error",
-                    f"{method} {path} -> {response.status_code}",
+                    "error",
+                    "http_response_error",
                     level="WARNING",
                     metadata={"method": method, "path": path, "status": response.status_code, "duration": round(duration, 3)},
                 )
@@ -112,10 +112,10 @@ class RequestLoggingMiddleware(BaseHTTPMiddleware):
             duration = time.time() - start_time
             print(f"[RESPONSE] method={method} path={path} status=500 duration={duration:.3f}s error={e}{user_tag}")
             log_event(
-                "response_error",
-                f"{method} {path} -> 500: {e}",
+                "error",
+                "http_response_error",
                 level="ERROR",
-                metadata={"method": method, "path": path, "status": 500, "duration": round(duration, 3)},
+                metadata={"method": method, "path": path, "status": 500, "duration": round(duration, 3), "error": str(e)},
             )
             raise
         finally:
@@ -140,14 +140,14 @@ print("[MIDDLEWARE] RequestLoggingMiddleware added")
 def handle_login(email, password):
     """Handle login attempt via Supabase"""
     print(f"[AUTH] Login attempt for: {email}")
-    log_event("login_attempt", f"Login attempt: {email}", metadata={"email": email})
+    log_event("auth", "login_attempt", metadata={"email": email})
     try:
         res = supabase.auth.sign_in_with_password({"email": email, "password": password})
         if res.session:
             print(f"[AUTH] Login successful: {email}")
             user_ctx = {"user_id": str(res.user.id), "email": email}
             log_event(
-                "login_success", f"Login success: {email}",
+                "auth", "login_success",
                 user_override=user_ctx, metadata={"email": email},
             )
             return (
@@ -158,7 +158,7 @@ def handle_login(email, password):
     except Exception as e:
         print(f"[AUTH] Login failed for {email}: {e}")
         log_event(
-            "login_failure", f"Login failed: {email} - {e}",
+            "auth", "login_failure",
             level="WARNING", metadata={"email": email, "error": str(e)},
         )
         return gr.update(), gr.update(value=f"❌ エラー: {str(e)}"), None
@@ -193,16 +193,11 @@ def get_current_user(request: Request):
         }
         
         print(f"[AUTH_CHECK] Success: user={user_dict['email']} role={user_dict['role']}")
-        log_event(
-            "auth_check", f"Authenticated: {user_dict['email']}",
-            user_override=user_dict,
-            metadata={"role": user_dict.get("role"), "org_name": user_dict.get("org_name")},
-        )
         return user_dict
         
     except Exception as e:
         print(f"[AUTH_CHECK] Failed: {e}")
-        log_event("auth_check_fail", f"Auth check failed: {e}", level="WARNING")
+        log_event("auth", "token_verify_fail", level="WARNING", metadata={"error": str(e)})
         return None
 
 # --- Create UI instances ---
@@ -262,7 +257,7 @@ async def logout():
     """Logout route - clear cookie and redirect to login"""
     user = get_user_context()
     print(f"[ROUTE] /logout accessed")
-    log_event("logout", "User logged out", user_override=user)
+    log_event("auth", "logout", user_override=user)
     response = RedirectResponse(url="/login/")
     response.delete_cookie("sb_access_token")
     return response