address test failures

withastro · Jan 18, 2024 · 55f0468 · 55f0468
1 parent 3252a2b
commit 55f0468
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/packages/integrations/node/src/serve-static.ts b/packages/integrations/node/src/serve-static.ts
@@ -37,6 +37,7 @@ export function createStaticHandler(app: NodeApp, options: Options) {
 						pathname = urlPath.slice(0, -1) + (urlQuery ? "?" + urlQuery : "");
 						res.statusCode = 301;
 						res.setHeader('Location', pathname);
+						return res.end();
 					} else pathname = urlPath;
 					// intentionally fall through
 				case "ignore":
@@ -52,16 +53,13 @@ export function createStaticHandler(app: NodeApp, options: Options) {
 						pathname = urlPath + '/' +(urlQuery ? "?" + urlQuery : "");
 						res.statusCode = 301;
 						res.setHeader('Location', pathname);
+						return res.end();
 					} else
 						pathname = urlPath;
 				break;
 			}
 			pathname = app.removeBase(pathname);
 
-			if (urlQuery && !pathname.includes('?')) {
-				pathname = pathname + '?' + urlQuery;
-			}
-
 			const stream = send(req, pathname, {
 				root: client,
 				dotfiles: pathname.startsWith('/.well-known/') ? 'allow' : 'deny',