edge-api: rewriter supports https://

joshbalfour · joshbalfour · commit c5a9366cf648 · 2023-10-25T11:51:07.000+01:00
diff --git a/packages/constructs/edge-api/src/lambdas/rewriter.ts b/packages/constructs/edge-api/src/lambdas/rewriter.ts
@@ -20,7 +20,7 @@ const middlewares: string[] = []
 const rewriteLocationHeader = (location: string, host: string) => {
   try {
     const url = new URL(location)
-    if (url.hostname !== host && domains.find((domain) => url.hostname.endsWith(domain))) {
+    if (url.hostname !== host && domains.find((domain) => url.hostname.endsWith(new URL(domain).hostname))) {
       url.hostname = host
     }
     return url.toString()

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ const middlewares: string[] = []`
`20`	`20`	`const rewriteLocationHeader = (location: string, host: string) => {`
`21`	`21`	`try {`
`22`	`22`	`const url = new URL(location)`
`23`		`- if (url.hostname !== host && domains.find((domain) => url.hostname.endsWith(domain))) {`
	`23`	`+ if (url.hostname !== host && domains.find((domain) => url.hostname.endsWith(new URL(domain).hostname))) {`
`24`	`24`	`url.hostname = host`
`25`	`25`	`}`
`26`	`26`	`return url.toString()`