fix: condition for exposing error via middleware

PrivateAIM · Oct 7, 2024 · e7a5fee · e7a5fee
1 parent 4b4fae4
commit e7a5fee
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/server-http-kit/src/middlewares/error.ts b/packages/server-http-kit/src/middlewares/error.ts
@@ -17,7 +17,7 @@ export function mountErrorMiddleware(router: Router) {
         const isServerError = error.statusCode >= 500 &&
             error.statusCode < 600;
 
-        if (isServerError) {
+        if (isServerError || error.logMessage) {
             useLogger().error(error);
 
             if (error.cause) {
@@ -62,7 +62,8 @@ export function mountErrorMiddleware(router: Router) {
         const exposeError = typeof error.expose === 'boolean' ?
             error.expose :
             !isServerError;
-        if (exposeError) {
+
+        if (!exposeError) {
             error.message = 'An internal server error occurred.';
         }
 
@@ -72,7 +73,7 @@ export function mountErrorMiddleware(router: Router) {
             statusCode: error.statusCode,
             code: `${error.code}`,
             message: error.message,
-            ...(isObject(error.data) && exposeError ? error.data : {}),
+            ...(exposeError && isObject(error.data) ? error.data : {}),
         };
     }));
 }