fix: add HttpException handling in GraphQL error formatter

Author: Ho-s

src/common/exceptions/exception.constant.ts

@@ -10,6 +10,7 @@ export const GRAPHQL_ERROR_CODES = [
 ];
 
 export const PRESERVED_STATUS_CODES = [
+  HttpStatus.BAD_REQUEST,
   HttpStatus.UNAUTHORIZED,
   HttpStatus.FORBIDDEN,
 ] as const;

src/common/exceptions/exception.format.ts

@@ -4,6 +4,7 @@ import { GraphQLError } from 'graphql';
 
 import { PRESERVED_STATUS_CODES } from './exception.constant';
 import {
+  getHttpExceptionCode,
   isBaseException,
   isGraphqlOriginalError,
   isHttpException,
@@ -26,6 +27,14 @@ const determineErrorCondition = (error: GraphQLError) => {
     };
   }
 
+  if (isHttpException(error.originalError)) {
+    const status = error.originalError.getStatus();
+    return {
+      errorStatus: status,
+      errorCode: getHttpExceptionCode(status),
+    };
+  }
+
   return {
     errorStatus: HttpStatus.INTERNAL_SERVER_ERROR,
     errorCode: 'INTERNAL_SERVER_ERROR',

src/common/exceptions/exception.util.ts

@@ -1,4 +1,4 @@
-import { HttpException } from '@nestjs/common';
+import { HttpException, HttpStatus } from '@nestjs/common';
 
 import { GraphQLErrorExtensions } from 'graphql';
 
@@ -23,3 +23,7 @@ export const isBaseException = (
 export const isHttpException = (error: unknown): error is HttpException => {
   return error instanceof HttpException;
 };
+
+export const getHttpExceptionCode = (status: number): string => {
+  return HttpStatus[status] || 'HTTP_ERROR';
+};