fix: jwt-auth.guard handleRequest wrong signature causing null.switchToHttp crash

This commit is contained in:
Nils-Johan Gynther
2026-04-30 13:34:29 +02:00
parent 0e525b0d17
commit cee62397fc
+8 -9
View File
@@ -1,4 +1,4 @@
import { Injectable, CanActivate, ExecutionContext, Logger } from '@nestjs/common'; import { Injectable, CanActivate, ExecutionContext, Logger, UnauthorizedException } from '@nestjs/common';
import { Observable } from 'rxjs'; import { Observable } from 'rxjs';
import { Reflector } from '@nestjs/core'; import { Reflector } from '@nestjs/core';
import { AuthGuard } from '@nestjs/passport'; import { AuthGuard } from '@nestjs/passport';
@@ -21,15 +21,14 @@ export class JwtAuthGuard extends AuthGuard('jwt') {
return super.canActivate(context); return super.canActivate(context);
} }
// Add logging for user ID in the JWT authentication guard // Log authenticated user ID for diagnostics
handleRequest(context: ExecutionContext, next: Function) { handleRequest<TUser = any>(err: any, user: TUser, info: any): TUser {
const request = context.switchToHttp().getRequest(); if (err || !user) {
const user = request.user; throw err || new UnauthorizedException();
}
if (user) { if (user) {
this.logger.log(`User ID: ${user.userId}, Username: ${user.username}`); this.logger.log(`Authenticated user ID: ${(user as any).userId}`);
} }
return user;
return next();
} }
} }