fix(nitro): ensure html is a string before injecting error handler

danielroe · danielroe · commit f70b70c97d82 · 2025-11-25T12:15:24.000Z
resolves #33748
diff --git a/packages/nitro-server/src/runtime/handlers/error.ts b/packages/nitro-server/src/runtime/handlers/error.ts
@@ -83,7 +83,7 @@ export default <NitroErrorHandler> async function errorhandler (error, event, {
   }
   setResponseStatus(event, res.status && res.status !== 200 ? res.status : defaultRes.status, res.statusText || defaultRes.statusText)
 
-  if (import.meta.dev && !import.meta.test) {
+  if (import.meta.dev && !import.meta.test && typeof html === 'string') {
     const prettyResponse = await defaultHandler(error, event, { json: false })
     return send(event, html.replace('</body>', `${generateErrorOverlayHTML(prettyResponse.body as string, { startMinimized: 300 <= statusCode && statusCode < 500 })}</body>`))
   }

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ export default <NitroErrorHandler> async function errorhandler (error, event, {`
`83`	`83`	`}`
`84`	`84`	`setResponseStatus(event, res.status && res.status !== 200 ? res.status : defaultRes.status, res.statusText \|\| defaultRes.statusText)`
`85`	`85`
`86`		`- if (import.meta.dev && !import.meta.test) {`
	`86`	`+ if (import.meta.dev && !import.meta.test && typeof html === 'string') {`
`87`	`87`	`const prettyResponse = await defaultHandler(error, event, { json: false })`
`88`	`88`	return send(event, html.replace('</body>', `${generateErrorOverlayHTML(prettyResponse.body as string, { startMinimized: 300 <= statusCode && statusCode < 500 })}</body>`))
`89`	`89`	`}`