Merge pull request #589 from CodinGame/fix-wrong-extension-host-worker-url

Do not force replace extension host worker url host

Author: CGNonofr

src/service-override/extensions.ts

@@ -314,7 +314,7 @@ export default function getServiceOverride(
     workerConfig != null
       ? {
           ...workerConfig,
-          url: changeUrlDomain(workerConfig.url, iframeAlternateDomain ?? globalThis.location?.href ?? import.meta.url)
+          url: changeUrlDomain(new URL(workerConfig.url, globalThis.location?.href ?? import.meta.url), iframeAlternateDomain)
         }
       : undefined
 

src/service-override/tools/url.ts

@@ -1,6 +1,6 @@
-export function changeUrlDomain(url: string, domain?: string): string {
+export function changeUrlDomain(url: string | URL, domain?: string): string {
   if (domain == null) {
-    return url
+    return url.toString()
   }
   const _url = new URL(url, domain)
   _url.host = new URL(domain).hostname