Don't consider path fragments with a single element to be a domain

browserpass · Sep 29, 2019 · bbf2e4d · bbf2e4d
1 parent ce56c8d
commit bbf2e4d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/helpers.js b/src/helpers.js
@@ -72,9 +72,11 @@ function prepareLogins(files, settings) {
                 allowFill: true
             };
             let domainInfo = pathToDomainInfo(storeId + "/" + login.login, settings.host);
-            login.domain = domainInfo ? domainInfo.hostname : null;
+            login.domain = domainInfo ? domainInfo.hostname : "";
             login.inCurrentHost =
-                host.hostname == login.domain || host.hostname.endsWith("." + login.domain);
+                host.hostname == login.domain || // host matches path component exactly
+                (login.domain.includes(".") && // path component is not a single level (e.g. com, net, local)...
+                    host.hostname.endsWith("." + login.domain)); // ...and the host ends with that path component
             if (domainInfo && domainInfo.port && domainInfo.port !== host.port) {
                 login.inCurrentHost = false;
             }