fix(server): handle the cases that res cannot be writable

web-infra-dev · Oct 24, 2024 · 8021750 · 8021750
1 parent 2b890bc
commit 8021750
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/packages/server/prod-server/src/apply.ts b/packages/server/prod-server/src/apply.ts
@@ -18,11 +18,10 @@ import { createLogger, isProd } from '@modern-js/utils';
 import type { ProdServerOptions } from './types';
 
 function getLogger() {
-  if (
-    process.env.DEBUG ||
-    (process.env.NODE_ENV === 'production' && !process.env.DISABLE_LOG)
-  ) {
-    return createLogger({ level: 'verbose' });
+  if (process.env.DEBUG || process.env.NODE_ENV === 'production') {
+    return createLogger({
+      level: (process.env.MODERN_SERVER_LOG_LEVEL as any) || 'verbose',
+    });
   } else {
     return createLogger();
   }

diff --git a/tests/package.json b/tests/package.json
@@ -6,7 +6,7 @@
   "main": "index.js",
   "scripts": {
     "test:rspack": "pnpm test:builder:rspack && pnpm test:framework && pnpm test:garfish:rspack",
-    "test:framework": "DISABLE_LOG=true jest",
+    "test:framework": "MODERN_SERVER_LOG_LEVEL=info jest",
     "test:builder:rspack": "cd e2e/builder && pnpm test:rspack",
     "test:garfish:rspack": "cd e2e/garfish && pnpm test:rspack",
     "test:module-tools": "cd integration && jest --testMatch **/module/**/*.test.ts",