fix: only set err.status on statusCode >= 200

cnpm · May 14, 2024 · 0403ef6 · 0403ef6
1 parent dbf5b52
commit 0403ef6
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/port/middleware/ErrorHandler.ts b/app/port/middleware/ErrorHandler.ts
@@ -31,7 +31,7 @@ export async function ErrorHandler(ctx: EggContext, next: Next) {
     }
 
     // http status, default is DEFAULT_SERVER_ERROR_STATUS
-    ctx.status = err.status || DEFAULT_SERVER_ERROR_STATUS;
+    ctx.status = (typeof err.status === 'number' && err.status >= 200) ? err.status : DEFAULT_SERVER_ERROR_STATUS;
     // don't log NotImplementedError
     if (ctx.status >= DEFAULT_SERVER_ERROR_STATUS && err.name !== 'NotImplementedError') {
       ctx.logger.error(err);