fix: use encodeURI only, drop URL.canParse per review

productdevbook · claude · productdevbook · commit 3a7536e54d3c · 2026-03-15T07:44:24.000+03:00
Per maintainer review:
- URL.canParse may not be available in all runtimes
- encodeURI handles both absolute and relative URLs correctly
- No need for conditional logic — encodeURI is sufficient

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/utils/response.ts b/src/utils/response.ts
@@ -44,7 +44,7 @@ export function redirect(
   status: number = 302,
   statusText?: string,
 ): HTTPResponse {
-  const encodedLoc = URL.canParse(location) ? new URL(location).href : encodeURI(location);
+  const encodedLoc = encodeURI(location);
   const body = /* html */ `<html><head><meta http-equiv="refresh" content="0; url=${encodedLoc}" /></head></html>`;
   return new HTTPResponse(body, {
     status,
