Merge pull request #22 from Visual-Regression-Tracker/73-register-error

73 register error

Author: pashidlos

src/app.module.ts

@@ -7,6 +7,8 @@ import { TestRunsModule } from './test-runs/test-runs.module';
 import { TestVariationsModule } from './test-variations/test-variations.module';
 import { PrismaService } from './prisma/prisma.service';
 import { ConfigModule } from '@nestjs/config';
+import { APP_FILTER } from '@nestjs/core';
+import { HttpPrismaExceptionFilter } from './http-prisma-exception.filter';
 
 @Module({
   imports: [
@@ -18,6 +20,12 @@ import { ConfigModule } from '@nestjs/config';
     TestRunsModule,
     TestVariationsModule,
   ],
-  providers: [PrismaService],
+  providers: [
+    PrismaService,
+    {
+      provide: APP_FILTER,
+      useClass: HttpPrismaExceptionFilter,
+    },
+  ],
 })
 export class AppModule {}

src/http-prisma-exception.filter.ts

@@ -0,0 +1,20 @@
+import { ExceptionFilter, Catch, ArgumentsHost, HttpException, HttpStatus } from '@nestjs/common';
+import { Request, Response } from 'express';
+import { PrismaClientKnownRequestError } from '@prisma/client';
+
+@Catch(PrismaClientKnownRequestError)
+export class HttpPrismaExceptionFilter implements ExceptionFilter {
+  catch(exception: PrismaClientKnownRequestError, host: ArgumentsHost) {
+    const ctx = host.switchToHttp();
+    const response = ctx.getResponse<Response>();
+    const request = ctx.getRequest<Request>();
+
+    response.status(HttpStatus.INTERNAL_SERVER_ERROR).json({
+      stack: exception.stack,
+      message: exception.message,
+      code: exception.code,
+      timestamp: new Date().toISOString(),
+      path: request.url,
+    });
+  }
+}

src/users/users.service.ts

@@ -25,19 +25,11 @@ export class UsersService {
       password: await this.authService.encryptPassword(createUserDto.password),
     };
 
-    try {
-      const userData = await this.prismaService.user.create({
-        data: user,
-      });
-
-      return new UserLoginResponseDto(userData, null);
-    } catch (err) {
-      if (err.original.constraint === 'user_email_key') {
-        throw new HttpException(`User with email '${err.errors[0].value}' already exists`, HttpStatus.CONFLICT);
-      }
+    const userData = await this.prismaService.user.create({
+      data: user,
+    });
 
-      throw new HttpException(err, HttpStatus.INTERNAL_SERVER_ERROR);
-    }
+    return new UserLoginResponseDto(userData, null);
   }
 
   async findOne(id: string): Promise<User> {