Filter out by error code + invert filtering (twentyhq#6432)

As title Instance of not working well. Co-authored-by: Charles Bochet <[email protected]>
ehconitin · Jul 29, 2024 · 515d6fb · 515d6fb
1 parent 9b28eeb
commit 515d6fb
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 13 deletions.
diff --git a/...ges/twenty-server/src/engine/core-modules/graphql/hooks/use-graphql-error-handler.hook.ts b/...ges/twenty-server/src/engine/core-modules/graphql/hooks/use-graphql-error-handler.hook.ts
@@ -64,12 +64,16 @@ export const useGraphQLErrorHandlerHook = <
             setResult,
           }) => {
             if (result.errors && result.errors.length > 0) {
-              const errorsToCapture = result.errors.reduce<BaseGraphQLError[]>(
-                (acc, error) => {
-                  if (!(error instanceof BaseGraphQLError)) {
-                    error = generateGraphQLErrorFromError(error);
-                  }
+              const originalErrors = result.errors.map((error) => {
+                const originalError = error.originalError;
 
+                return originalError instanceof BaseGraphQLError
+                  ? error.originalError
+                  : generateGraphQLErrorFromError(error);
+              });
+
+              const errorsToCapture = originalErrors.reduce<BaseGraphQLError[]>(
+                (acc, error) => {
                   if (shouldCaptureException(error)) {
                     acc.push(error);
                   }
@@ -95,7 +99,7 @@ export const useGraphQLErrorHandlerHook = <
                 errorsToCapture.map((err, i) => addEventId(err, eventIds?.[i]));
               }
 
-              const nonCapturedErrors = result.errors.filter(
+              const nonCapturedErrors = originalErrors.filter(
                 (error) => !errorsToCapture.includes(error),
               );
 

diff --git a/...ty-server/src/engine/core-modules/graphql/utils/generate-graphql-error-from-error.util.ts b/...ty-server/src/engine/core-modules/graphql/utils/generate-graphql-error-from-error.util.ts
@@ -14,5 +14,5 @@ export const generateGraphQLErrorFromError = (error: Error) => {
     graphqlError.extensions['response'] = error.message;
   }
 
-  return error;
+  return graphqlError;
 };
diff --git a/...ages/twenty-server/src/engine/core-modules/graphql/utils/should-capture-exception.util.ts b/...ages/twenty-server/src/engine/core-modules/graphql/utils/should-capture-exception.util.ts
@@ -3,7 +3,7 @@ import {
   ErrorCode,
 } from 'src/engine/core-modules/graphql/utils/graphql-errors.util';
 
-export const graphQLErrorCodesToFilter = [
+export const graphQLErrorCodesToFilterOut = [
   ErrorCode.GRAPHQL_VALIDATION_FAILED,
   ErrorCode.UNAUTHENTICATED,
   ErrorCode.FORBIDDEN,
@@ -15,12 +15,13 @@ export const graphQLErrorCodesToFilter = [
 ];
 
 export const shouldCaptureException = (exception: Error): boolean => {
-  if (
-    exception instanceof BaseGraphQLError &&
-    graphQLErrorCodesToFilter.includes(exception?.extensions?.code)
-  ) {
+  if (!(exception instanceof BaseGraphQLError)) {
     return true;
   }
 
-  return false;
+  if (graphQLErrorCodesToFilterOut.includes(exception?.extensions?.code)) {
+    return false;
+  }
+
+  return true;
 };