feat: Enhance error logging in API handlers

routes/api/eh/image_limit.ts

@@ -3,6 +3,9 @@ import { User, UserPermission } from "../../../db.ts";
 import { get_task_manager } from "../../../server.ts";
 import { EHImageLimit } from "../../../server/eh.ts";
 import { return_data, return_error } from "../../../server/utils.ts";
+import { base_logger } from "../../../utils/logger.ts";
+
+const logger = base_logger.get_logger("api-eh-image-limit");
 
 export const handler: Handlers = {
     async GET(_req, ctx) {
@@ -25,7 +28,8 @@ export const handler: Handlers = {
                 current: re.current_image_limit,
             });
         } catch (e) {
-            return return_error(500, e.message);
+            logger.error("Failed to fetch image limit:", e);
+            return return_error(500, "Internal server error.");
         }
     },
 };

routes/api/eh/metadata.ts

@@ -10,6 +10,9 @@ import {
     return_data,
     return_error,
 } from "../../../server/utils.ts";
+import { base_logger } from "../../../utils/logger.ts";
+
+const logger = base_logger.get_logger("api-eh-metadata");
 
 export const handler: Handlers = {
     async GET(req, ctx) {
@@ -75,9 +78,11 @@ export const handler: Handlers = {
                     }
                 }
             } catch (e) {
+                logger.error("Failed to fetch metadata:", e);
+                const mes = e instanceof Error ? e.message : e;
                 return return_error(
                     4,
-                    `Failed to fetch metadata: ${e.message}`,
+                    `Failed to fetch metadata: ${mes}`,
                 );
             }
         }

routes/api/file/random.ts

@@ -8,6 +8,9 @@ import pbkdf2Hmac from "pbkdf2-hmac";
 import { encodeBase64 as encode } from "@std/encoding/base64";
 import { return_data } from "../../../server/utils.ts";
 import { extname } from "@std/path";
+import { base_logger } from "../../../utils/logger.ts";
+
+const logger = base_logger.get_logger("api-file-random");
 
 export const handler: Handlers = {
     async GET(req, ctx) {
@@ -102,7 +105,8 @@ export const handler: Handlers = {
                     gids?.add(d.gid);
                 });
             } catch (e) {
-                return new Response(e.message, { status: 400 });
+                logger.error("Failed to search with meilisearch:", e);
+                return new Response("Search failed.", { status: 400 });
             }
         }
         const f = m.db.get_random_file(is_nsfw, is_ad, gids);

routes/api/token.ts

@@ -7,6 +7,9 @@ import pbkdf2Hmac from "pbkdf2-hmac";
 import isEqual from "lodash/isEqual";
 import type { Token, User } from "../../db.ts";
 import { Mutex } from "async/mutex.ts";
+import { base_logger } from "../../utils/logger.ts";
+
+const logger = base_logger.get_logger("api-token");
 
 const USER_PASSWORD_ERROR = "Incorrect username or password.";
 
@@ -189,7 +192,8 @@ export const handler: Handlers = {
                 ),
             );
         } catch (e) {
-            return return_error(500, e.message);
+            logger.error("Failed to update token:", e);
+            return return_error(500, "Internal server error");
         }
     },
 };