fix: logging system

Author: csuvajit

libs/interceptors/http-exceptions-filter.ts

@@ -53,13 +53,12 @@ export class HttpExceptionsFilter implements ExceptionFilter {
       ip: req.ip,
       ...mapKeys(
         req.user || {
-          userId: 'unauthorized',
           body: pick(req.body || {}, ['passKey']),
           headers: pick(req.headers || {}, ['x-access-token', 'authorization']),
         },
         (_, key) => `user.${key}`,
       ),
-      userAgent: req.headers['user-agent'],
+      user_agent: req.headers['user-agent'],
     });
 
     return res.status(status).json({

libs/interceptors/http-logging.middleware.ts

@@ -1,6 +1,10 @@
 import { ConsoleLogger, Injectable, Logger, NestMiddleware } from '@nestjs/common';
+import * as Sentry from '@sentry/nestjs';
 import { NextFunction, Request, Response } from 'express';
+import mapKeys from 'lodash/mapKeys';
+import pick from 'lodash/pick';
 import moment from 'moment';
+import { fallbackUser } from '../../src/auth';
 
 export class CustomLogger extends ConsoleLogger {
   getTimestamp(): string {
@@ -36,6 +40,22 @@ export class HttpLoggingMiddleware implements NestMiddleware {
 
     const logType = statusCode >= 500 ? 'error' : statusCode >= 400 ? 'warn' : 'debug';
     this.logger[logType](logMessage, method);
+
+    if (req.user?.userId !== fallbackUser.userId) {
+      Sentry.logger.info(`[${req.method}] ${statusCode} ${req.url}`, {
+        status: statusCode,
+        method: req.method,
+        path: req.url,
+        ip: req.ip,
+        ...mapKeys(
+          req.user || {
+            body: pick(req.body || {}, ['passKey']),
+          },
+          (_, key) => `user.${key}`,
+        ),
+        user_agent: req.headers['user-agent'],
+      });
+    }
   }
 
   private formatIp(ip?: string): string {

src/auth/auth.controller.ts

@@ -55,7 +55,7 @@ export class AuthController {
   @ApiExcludeEndpoint()
   async loginWithTurnstile(
     @Req() req: Request,
-    @Res() res: Response,
+    @Res({ passthrough: true }) res: Response,
     @Body() body: TurnstileLoginDto,
   ): Promise<LoginOkDto> {
     const remoteIp = (req.headers['cf-connecting-ip'] || req.ip) as string;
@@ -66,10 +66,10 @@ export class AuthController {
       secure: true,
       sameSite: 'strict',
       signed: true,
-      maxAge: 5 * 60 * 60 * 1000,
+      maxAge: 5 * 60 * 1000,
     });
 
-    return res.json(result) as unknown as LoginOkDto;
+    return result;
   }
 
   @Get('/handoff/:token')

src/exports/exports.controller.ts

@@ -10,7 +10,7 @@ import { ExportsService } from './exports.service';
 export class ExportsController {
   constructor(private readonly exportsService: ExportsService) {}
 
-  @Cache(10 * 60 * 1000)
+  @Cache(10 * 60)
   @Post('/members')
   exportClanMembers(@Body() body: ExportMembersInput): Promise<ExportMembersOutputDto> {
     return this.exportsService.exportClanMembers(body);