fix: some refactor

Author: filippovskii09

backend/src/auth/strategies/jwt.strategy.ts

@@ -1,5 +1,5 @@
 import { ConfigService } from '@nestjs/config';
-import { Injectable, Logger } from '@nestjs/common';
+import { Injectable } from '@nestjs/common';
 import { PassportStrategy } from '@nestjs/passport';
 
 import { ExtractJwt, Strategy } from 'passport-jwt';
@@ -9,8 +9,6 @@ import { JwtPayload } from '../types';
 
 @Injectable()
 export class JwtStrategy extends PassportStrategy(Strategy) {
-  private readonly logger = new Logger(JwtStrategy.name);
-
   constructor(private readonly configService: ConfigService) {
     const secret = configService.getOrThrow<string>(JWT_ACCESS_TOKEN_SECRET);
     super({
@@ -21,7 +19,6 @@ export class JwtStrategy extends PassportStrategy(Strategy) {
   }
 
   async validate(payload: JwtPayload) {
-    this.logger.log(`Validating JWT token: ${JSON.stringify(payload)}`);
     return { id: payload.sub, email: payload.email };
   }
 }

backend/src/workouts/workouts.controller.ts

@@ -8,6 +8,7 @@ import {
   ParseUUIDPipe,
   Logger,
   UseGuards,
+  UnauthorizedException,
 } from '@nestjs/common';
 
 import { JwtAuthGuard } from '@src/auth/guards';
@@ -30,7 +31,7 @@ export class WorkoutsController {
   ) {
     if (!userId) {
       this.logger.error(ResponseMessages.User.IdNotFound, userId);
-      throw new Error(ResponseMessages.User.IdNotFound);
+      throw new UnauthorizedException(ResponseMessages.User.IdNotFound);
     }
     return await this.workoutsService.create(CreateWorkoutDto, userId);
   }
@@ -40,7 +41,7 @@ export class WorkoutsController {
   async findAll(@CurrentUser('id') userId: string) {
     if (!userId) {
       this.logger.error(ResponseMessages.User.IdNotFound, userId);
-      throw new Error(ResponseMessages.User.IdNotFound);
+      throw new UnauthorizedException(ResponseMessages.User.IdNotFound);
     }
     return await this.workoutsService.findAll(userId);
   }