feat(be): implement GraphQL Exceptions

backend/apps/admin/src/admin.module.ts

@@ -12,6 +12,7 @@ import {
   GroupLeaderGuard
 } from '@libs/auth'
 import { CacheConfigService } from '@libs/cache'
+import { apolloErrorFormatter } from '@libs/exception'
 import { pinoLoggerModuleOption } from '@libs/logger'
 import { PrismaModule } from '@libs/prisma'
 import { AdminController } from './admin.controller'
@@ -34,7 +35,8 @@ import { UserModule } from './user/user.module'
       driver: ApolloDriver,
       autoSchemaFile: 'schema.gql',
       sortSchema: true,
-      introspection: true
+      introspection: true,
+      formatError: apolloErrorFormatter
     }),
     CacheModule.registerAsync({
       isGlobal: true,

backend/apps/admin/src/contest/contest.resolver.ts

@@ -1,10 +1,7 @@
 import {
-  BadRequestException,
   InternalServerErrorException,
   Logger,
-  NotFoundException,
-  ParseBoolPipe,
-  UnprocessableEntityException
+  ParseBoolPipe
 } from '@nestjs/common'
 import { Args, Context, Int, Mutation, Query, Resolver } from '@nestjs/graphql'
 import { AuthenticatedRequest, UseRolesGuard } from '@libs/auth'
@@ -53,10 +50,11 @@ export class ContestResolver {
         input
       )
     } catch (error) {
-      if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
-      } else if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
+      if (
+        error instanceof UnprocessableDataException ||
+        error instanceof EntityNotExistException
+      ) {
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -71,10 +69,11 @@ export class ContestResolver {
     try {
       return await this.contestService.updateContest(groupId, input)
     } catch (error) {
-      if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
-      } else if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
+      if (
+        error instanceof EntityNotExistException ||
+        error instanceof UnprocessableDataException
+      ) {
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -90,7 +89,7 @@ export class ContestResolver {
       return await this.contestService.deleteContest(groupId, contestId)
     } catch (error) {
       if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -114,10 +113,11 @@ export class ContestResolver {
         contestId
       )
     } catch (error) {
-      if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
-      } else if (error instanceof ConflictFoundException) {
-        throw new BadRequestException(error.message)
+      if (
+        error instanceof EntityNotExistException ||
+        error instanceof ConflictFoundException
+      ) {
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -137,7 +137,7 @@ export class ContestResolver {
       )
     } catch (error) {
       if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -158,7 +158,7 @@ export class ContestResolver {
       )
     } catch (error) {
       if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()

backend/apps/admin/src/group/group.resolver.ts

@@ -1,9 +1,4 @@
-import {
-  ConflictException,
-  ForbiddenException,
-  InternalServerErrorException,
-  Logger
-} from '@nestjs/common'
+import { InternalServerErrorException, Logger } from '@nestjs/common'
 import { Args, Int, Query, Mutation, Resolver, Context } from '@nestjs/graphql'
 import { Group } from '@generated'
 import { Role } from '@prisma/client'
@@ -34,7 +29,7 @@ export class GroupResolver {
       return await this.groupService.createGroup(input, req.user.id)
     } catch (error) {
       if (error instanceof DuplicateFoundException) {
-        throw new ConflictException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -66,10 +61,11 @@ export class GroupResolver {
     try {
       return await this.groupService.updateGroup(id, input)
     } catch (error) {
-      if (error instanceof DuplicateFoundException) {
-        throw new ConflictException(error.message)
-      } else if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
+      if (
+        error instanceof DuplicateFoundException ||
+        error instanceof ForbiddenAccessException
+      ) {
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -85,7 +81,7 @@ export class GroupResolver {
       return await this.groupService.deleteGroup(id, req.user)
     } catch (error) {
       if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -98,7 +94,7 @@ export class GroupResolver {
       return await this.groupService.issueInvitation(id)
     } catch (error) {
       if (error instanceof ConflictFoundException) {
-        throw new ConflictException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()

backend/apps/admin/src/problem/problem.resolver.ts

@@ -1,6 +1,4 @@
 import {
-  ConflictException,
-  ForbiddenException,
   InternalServerErrorException,
   Logger,
   NotFoundException,
@@ -50,7 +48,7 @@ export class ProblemResolver {
       )
     } catch (error) {
       if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
+        throw error.convert2HTTPException()
       } else if (
         error instanceof Prisma.PrismaClientKnownRequestError &&
         error.code === 'P2003'
@@ -78,7 +76,7 @@ export class ProblemResolver {
       )
     } catch (error) {
       if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -126,16 +124,17 @@ export class ProblemResolver {
     try {
       return await this.problemService.updateProblem(input, groupId)
     } catch (error) {
-      if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
+      if (
+        error instanceof UnprocessableDataException ||
+        error instanceof ConflictFoundException
+      ) {
+        throw error.convert2HTTPException()
       } else if (error instanceof Prisma.PrismaClientKnownRequestError) {
         if (error.name == 'NotFoundError') {
           throw new NotFoundException(error.message)
         } else if (error.code === 'P2003') {
           throw new UnprocessableEntityException(error.message)
         }
-      } else if (error instanceof ConflictFoundException) {
-        throw new ConflictException(error.message)
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -176,10 +175,11 @@ export class ProblemResolver {
     try {
       return this.problemService.getWorkbookProblems(groupId, workbookId)
     } catch (error) {
-      if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
-      } else if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
+      if (
+        error instanceof UnprocessableDataException ||
+        error instanceof ForbiddenAccessException
+      ) {
+        throw error.convert2HTTPException()
       } else if (error.code == 'P2025') {
         throw new EntityNotExistException(error.message)
       }
@@ -208,10 +208,11 @@ export class ProblemResolver {
         orders
       )
     } catch (error) {
-      if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
-      } else if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
+      if (
+        error instanceof UnprocessableDataException ||
+        error instanceof ForbiddenAccessException
+      ) {
+        throw error.convert2HTTPException()
       } else if (error.code == 'P2025') {
         throw new EntityNotExistException(error.message)
       }
@@ -234,10 +235,11 @@ export class ProblemResolver {
     try {
       return this.problemService.getContestProblems(groupId, contestId)
     } catch (error) {
-      if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
-      } else if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
+      if (
+        error instanceof UnprocessableDataException ||
+        error instanceof ForbiddenAccessException
+      ) {
+        throw error.convert2HTTPException()
       } else if (error.code == 'P2025') {
         throw new EntityNotExistException(error.message)
       }
@@ -265,10 +267,11 @@ export class ProblemResolver {
         orders
       )
     } catch (error) {
-      if (error instanceof UnprocessableDataException) {
-        throw new UnprocessableEntityException(error.message)
-      } else if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
+      if (
+        error instanceof UnprocessableDataException ||
+        error instanceof ForbiddenAccessException
+      ) {
+        throw error.convert2HTTPException()
       } else if (error.code == 'P2025') {
         throw new EntityNotExistException(error.message)
       }

backend/apps/client/src/announcement/announcement.controller.ts

@@ -2,7 +2,6 @@ import {
   Controller,
   Get,
   Logger,
-  NotFoundException,
   InternalServerErrorException,
   Query,
   BadRequestException
@@ -39,7 +38,7 @@ export class AnnouncementController {
       }
     } catch (error) {
       if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()

backend/apps/client/src/auth/auth.controller.ts

@@ -52,7 +52,7 @@ export class AuthController {
       this.setJwtResponse(res, jwtTokens)
     } catch (error) {
       if (error instanceof UnidentifiedException) {
-        throw new UnauthorizedException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException('Login failed')
@@ -87,7 +87,7 @@ export class AuthController {
       this.setJwtResponse(res, newJwtTokens)
     } catch (error) {
       if (error instanceof InvalidJwtTokenException) {
-        throw new UnauthorizedException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException('Failed to reissue tokens')
@@ -114,7 +114,7 @@ export class AuthController {
       return await this.authService.githubLogin(res, githubUser)
     } catch (error) {
       if (error instanceof UnidentifiedException) {
-        throw new UnauthorizedException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException('Login failed')
@@ -142,7 +142,7 @@ export class AuthController {
       return await this.authService.kakaoLogin(res, kakaoUser)
     } catch (error) {
       if (error instanceof UnidentifiedException) {
-        throw new UnauthorizedException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException('Login failed')

backend/apps/client/src/contest/contest.controller.ts

@@ -8,7 +8,6 @@ import {
   Get,
   Query,
   Logger,
-  ConflictException,
   DefaultValuePipe
 } from '@nestjs/common'
 import { Prisma } from '@prisma/client'
@@ -119,7 +118,7 @@ export class ContestController {
       ) {
         throw new NotFoundException(error.message)
       } else if (error instanceof ConflictFoundException) {
-        throw new ConflictException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException(error.message)

backend/apps/client/src/group/group.controller.ts

@@ -1,9 +1,7 @@
 import {
-  ConflictException,
   Controller,
   DefaultValuePipe,
   Delete,
-  ForbiddenException,
   Get,
   InternalServerErrorException,
   Logger,
@@ -95,7 +93,7 @@ export class GroupController {
       ) {
         throw new NotFoundException('Invalid invitation')
       } else if (error instanceof EntityNotExistException) {
-        throw new NotFoundException(error.message)
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -120,10 +118,11 @@ export class GroupController {
         error.name === 'NotFoundError'
       ) {
         throw new NotFoundException(error.message)
-      } else if (error instanceof ForbiddenAccessException) {
-        throw new ForbiddenException(error.message)
-      } else if (error instanceof ConflictFoundException) {
-        throw new ConflictException(error.message)
+      } else if (
+        error instanceof ForbiddenAccessException ||
+        error instanceof ConflictFoundException
+      ) {
+        throw error.convert2HTTPException()
       }
       this.logger.error(error)
       throw new InternalServerErrorException()
@@ -140,7 +139,7 @@ export class GroupController {
       return await this.groupService.leaveGroup(req.user.id, groupId)
     } catch (error) {
       if (error instanceof ConflictFoundException) {
-        throw new ConflictException(error.message)
+        throw error.convert2HTTPException()
       }
 
       this.logger.error(error)