Merge pull request #64 from mapeveri/refactor/fix-refresh-controller

Refactor: Improvements error controller

src/languages/infrastructure/ui/api/v1/controllers/auth/meGetController.ts

@@ -1,6 +1,6 @@
 import { Request } from 'express';
 import FindUserQuery from '@src/languages/application/user/query/find/findUserQuery';
-import { Controller, Get, HttpCode, Inject, Req, UseGuards } from '@nestjs/common';
+import { Controller, Get, HttpCode, HttpException, HttpStatus, Inject, Req, UseGuards } from '@nestjs/common';
 import { JwtAuthGuard } from '@src/shared/infrastructure/nestjs/guards/JwtAuthGuard';
 import MeGetResponseDto from './meGetResponseDto';
 import {
@@ -27,7 +27,7 @@ export default class MeGetController {
   async run(@Req() request: Request): Promise<MeGetResponseDto> {
     const userId = request.user?.id;
     if (!userId) {
-      throw new Error('Invalid user');
+      throw new HttpException('Invalid user', HttpStatus.FORBIDDEN);
     }
 
     const data = await this.queryBus.ask(new FindUserQuery(userId));

src/languages/infrastructure/ui/api/v1/controllers/auth/refreshTokenPostController.ts

@@ -1,4 +1,4 @@
-import { Body, Controller, HttpCode, Inject, Post } from '@nestjs/common';
+import { Body, Controller, HttpCode, HttpException, HttpStatus, Inject, Post } from '@nestjs/common';
 import { JwtService } from '@nestjs/jwt';
 import LoginPostResponseDto from './loginPostResponseDto';
 import { ApiBadRequestResponse, ApiInternalServerErrorResponse, ApiOkResponse, ApiTags } from '@nestjs/swagger';
@@ -20,7 +20,7 @@ export default class RefreshTokenPostController {
   async run(@Body() payload: RefreshTokenPostDto): Promise<RefreshTokenPostResponseDto> {
     const decodedRefreshToken = this.jwtService.verify(payload.refreshToken);
     if (decodedRefreshToken.revoked) {
-      throw new Error('Token revoked');
+      throw new HttpException('Token revocado', HttpStatus.FORBIDDEN);
     }
 
     const user = await this.queryBus.ask(new FindUserQuery(decodedRefreshToken.id));