fix: refactor logging

brinestone · Jan 14, 2025 · b75ea81 · b75ea81
1 parent 2b19ede
commit b75ea81
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/server/handlers/auth.mts b/server/handlers/auth.mts
@@ -23,7 +23,7 @@ passport.use(new GoogleStrategy({
   callbackURL: `${process.env['ORIGIN']}/api/auth/google/callback`,
 }, async (accessToken: string, __: string, profile: Profile, done: VerifyCallback) => {
   try {
-    logger.info('completing oauth2 request');
+    logger.info('completing oauth2 request', { profile: { id: profile.id, provider: 'google', } });
     const db = useUsersDb();
     let existingUser = await db.query.users.findFirst({
       where: (user, { eq }) => eq(user.credentials, profile.id)

diff --git a/server/middleware/logger.mts b/server/middleware/logger.mts
@@ -1,5 +1,5 @@
 import { NextFunction, Request, Response } from 'express';
-import defaultLogger                       from '@logger/common';
+import defaultLogger from '@logger/common';
 
 export default function logger(req: Request, res: Response, next: NextFunction) {
   const start = Date.now();
@@ -8,11 +8,11 @@ export default function logger(req: Request, res: Response, next: NextFunction)
     const diff = (now - start) / 1000
     const message = `${req.method} ${req.originalUrl} -> ${res.statusCode} | ${diff}s`;
     if (res.statusCode < 400) {
-      defaultLogger.verbose(message, ['duration', diff]);
+      defaultLogger.verbose(message, { duration: diff });
     } else if (res.statusCode >= 400 && res.statusCode < 500) {
-      defaultLogger.warn(message, ['duration', diff]);
+      defaultLogger.warn(message, { duration: diff });
     } else {
-      defaultLogger.error(message, ['duration', diff]);
+      defaultLogger.error(message, { duration: diff });
     }
   });
   next();