Allowlist localhost for amp-worker and extensions in localDev (amppro…

…ject#39424)
danielrozenberg · Sep 1, 2023 · 1ae91e0 · 1ae91e0
1 parent 4856d04
commit 1ae91e0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/service/extension-script.js b/src/service/extension-script.js
@@ -164,7 +164,8 @@ export function createExtensionScript(win, extensionId, version) {
       // Only allow trusted URLs
       if (
         regexURL.test(url) ||
-        (getMode().test && testRegexURL.test(new URL(url).hostname)) ||
+        ((getMode().test || getMode().localDev) &&
+          testRegexURL.test(new URL(url).hostname)) ||
         new URL(url).host === 'fonts.googleapis.com'
       ) {
         return url;

diff --git a/src/web-worker/amp-worker.js b/src/web-worker/amp-worker.js
@@ -82,7 +82,7 @@ class AmpWorker {
           /^https:\/\/([a-zA-Z0-9_-]+\.)?cdn\.ampproject\.org(\/.*)?$/;
 
         if (
-          (regexURL.test(url) || getMode().test) &&
+          (regexURL.test(url) || getMode().test || getMode().localDev) &&
           (url.endsWith('ww.js') ||
             url.endsWith('ww.min.js') ||
             url.endsWith('ww.mjs') ||