Revert "fix(extension): #107: allow http for localhost (#129)"

This reverts commit 8141aa4.

apps/extension/public/manifest.json

@@ -16,7 +16,7 @@
   },
   "content_scripts": [
     {
-      "matches": ["https://*/*", "http://localhost:*/*"],
+      "matches": ["https://*/*"],
       "js": [
         "injected-connection-port.js",
         "injected-disconnect-listener.js",
@@ -25,7 +25,7 @@
       "run_at": "document_start"
     },
     {
-      "matches": ["https://*/*", "http://localhost:*/*"],
+      "matches": ["https://*/*"],
       "js": ["injected-penumbra-global.js"],
       "run_at": "document_start",
       "world": "MAIN"

apps/extension/src/senders/validate.ts

@@ -6,12 +6,12 @@ type ValidSender = chrome.runtime.MessageSender & {
   frameId: 0;
   documentId: string;
   tab: chrome.tabs.Tab & { id: number };
-  origin: string;
-  url: string;
-};
 
-const isHttpLocalhost = (url: URL): boolean =>
-  url.protocol === 'http:' && url.hostname === 'localhost';
+  // the relationship between origin and url is pretty complex.
+  // just rely on the browser's tools.
+  origin: `${ValidProtocol}//${string}`;
+  url: `${ValidProtocol}//${string}/${string}`;
+};
 
 export const assertValidSender = (sender?: chrome.runtime.MessageSender) => {
   if (!sender) {
@@ -34,13 +34,7 @@ export const assertValidSender = (sender?: chrome.runtime.MessageSender) => {
   if (parsedOrigin.origin !== sender.origin) {
     throw new Error('Sender origin is invalid');
   }
-
-  if (
-    !(
-      parsedOrigin.protocol in ValidProtocol ||
-      (globalThis.__DEV__ && isHttpLocalhost(parsedOrigin))
-    )
-  ) {
+  if (!(parsedOrigin.protocol in ValidProtocol)) {
     throw new Error(`Sender protocol is not ${Object.values(ValidProtocol).join(',')}`);
   }