fix: stronger pattern binding (could be bad?)

nobkd · Mar 26, 2023 · 56fef8f · 56fef8f
1 parent fd65e8e
commit 56fef8f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/manifest.json b/manifest.json
@@ -12,10 +12,10 @@
     "permissions": [
         "webRequest",
         "webRequestBlocking",
-        "http://*/maps/embed*",
-        "https://*/maps/embed*",
-        "http://*/maps*output=embed*",
-        "https://*/maps*output=embed*"
+        "http://*/maps/embed?*",
+        "https://*/maps/embed?*",
+        "http://*/maps?*output=embed*",
+        "https://*/maps?*output=embed*"
     ],
     "background": {
         "scripts": [

diff --git a/replace.js b/replace.js
@@ -1,15 +1,15 @@
 "use strict";
 
 const patterns = [
-    'http://*/maps/embed*',
-    'https://*/maps/embed*',
-    'http://*/maps*output=embed*',
-    'https://*/maps*output=embed*'
+    'http://*/maps/embed?*',
+    'https://*/maps/embed?*',
+    'http://*/maps?*output=embed*',
+    'https://*/maps?*output=embed*'
 ];
 
 const gLocales = ['com', 'de']; // TODO: collect more locales
 const matcher = new RegExp(
-    `^(https?:\/\/)?(maps\.google\.(${gLocales.join('|')})\/maps\/?\?.*output=embed|(www\.)?google\.(${gLocales.join('|')})\/maps\/embed\/?\?)`
+    `^(https?:\/\/)?(maps\.google\.(${gLocales.join('|')})\/maps\?.*output=embed|(www\.)?google\.(${gLocales.join('|')})\/maps\/embed\?)`
 );