fix(node): Safely create requests (#10285)

* fix(node): Wrap request creation in try catch * chore: changeset
withastro · Mar 1, 2024 · 0781eb6 · 0781eb6
1 parent fea77cf
commit 0781eb6
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/packages/integrations/node/src/serve-app.ts b/packages/integrations/node/src/serve-app.ts
@@ -8,7 +8,15 @@ import type { RequestHandler } from './types.js';
  */
 export function createAppHandler(app: NodeApp): RequestHandler {
 	return async (req, res, next, locals) => {
-		const request = NodeApp.createRequest(req);
+		let request;
+		try {
+			request = NodeApp.createRequest(req);
+		} catch (err) {
+			res.statusCode = 500;
+			res.end('Internal Server Error');
+			return;
+		}
+
 		const routeData = app.match(request);
 		if (routeData) {
 			const response = await app.render(request, {