fix(sendRedirect): always encode location uri

pi0 · pi0 · commit 01476acb98a2 · 2022-08-30T10:51:24.000+02:00
diff --git a/src/utils/response.ts b/src/utils/response.ts
@@ -24,13 +24,14 @@ export function defaultContentType (event: CompatibilityEvent, type?: string) {
 }
 
 export function sendRedirect (event: CompatibilityEvent, location: string, code = 302) {
+  const encodedLoc = encodeURI(location)
   event.res.statusCode = code
-  event.res.setHeader('Location', location)
-  // minimal html document that redirects on client side
+  event.res.setHeader('Location', encodedLoc)
+  // Minimal html document that redirects on client side
   const html = `<!DOCTYPE html>
 <html>
-  <head><meta http-equiv="refresh" content="0; url=${encodeURI(location)}"></head>
-  <body>Redirecting to <a href=${JSON.stringify(location)}>${encodeURI(location)}</a></body>
+  <head><meta http-equiv="refresh" content="0; url=${encodedLoc}"></head>
+  <body>Redirecting to <a href=${JSON.stringify(encodedLoc)}>${encodedLoc}</a></body>
 </html>`
   return send(event, html, MIMES.html)
 }
