From 4fcdfbff7dfaa1fdc2e71abd39edd58c93e06c5f Mon Sep 17 00:00:00 2001 From: martmull Date: Tue, 21 May 2024 11:31:03 +0200 Subject: [PATCH] Fix unhandled exception (#5474) Solves exception.getStatus is not a function error logs in twenty-server Catch all errors in order to have no error log at all --- .../src/utils/apply-cors-to-exceptions.ts | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/packages/twenty-server/src/utils/apply-cors-to-exceptions.ts b/packages/twenty-server/src/utils/apply-cors-to-exceptions.ts index 1b17fd2ab..0dd3a5cc1 100644 --- a/packages/twenty-server/src/utils/apply-cors-to-exceptions.ts +++ b/packages/twenty-server/src/utils/apply-cors-to-exceptions.ts @@ -1,4 +1,9 @@ -import { ExceptionFilter, Catch, ArgumentsHost } from '@nestjs/common'; +import { + ExceptionFilter, + Catch, + ArgumentsHost, + HttpException, +} from '@nestjs/common'; import { Response } from 'express'; @@ -25,6 +30,9 @@ export class ApplyCorsToExceptions implements ExceptionFilter { 'Origin, X-Requested-With, Content-Type, Accept', ); - response.status(exception.getStatus()).json(exception.response); + const status = + exception instanceof HttpException ? exception.getStatus() : 500; + + response.status(status).json(exception.response); } }