Skip to content

Commit 94d0080

Browse files
authored
perf: only register time middleware when debug logging is enabled (#23621)
1 parent 89574f6 commit 94d0080

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

‎packages/vite/src/node/server/index.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ import {
9494
serveRawFsMiddleware,
9595
serveStaticMiddleware,
9696
} from './middlewares/static'
97-
import { timeMiddleware } from './middlewares/time'
97+
import { isTimeDebugEnabled, timeMiddleware } from './middlewares/time'
9898
import {
9999
cachedTransformMiddleware,
100100
transformMiddleware,
@@ -989,7 +989,7 @@ export async function _createServer(
989989
// Pre applied internal middlewares ------------------------------------------
990990

991991
// request timer
992-
if (process.env.DEBUG) {
992+
if (isTimeDebugEnabled) {
993993
middlewares.use(timeMiddleware(root))
994994
}
995995

‎packages/vite/src/node/server/middlewares/time.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@ import { createDebugger, prettifyUrl, timeFrom } from '../../utils'
44

55
const logTime = createDebugger('vite:time')
66

7+
export const isTimeDebugEnabled: boolean = logTime !== undefined
8+
79
export function timeMiddleware(root: string): Connect.NextHandleFunction {
810
// Keep the named function. The name is visible in debug logs via `DEBUG=connect:dispatcher ...`
911
return function viteTimeMiddleware(req, res, next) {

0 commit comments

Comments
 (0)