Skip to content

Commit

Permalink
Revert "Add an injection blacklist. Addresses #483"
Browse files Browse the repository at this point in the history
This reverts commit 5bab058.

Addressed more comprehensively with a410570
  • Loading branch information
adomasven committed Jun 27, 2024
1 parent a410570 commit f0d454d
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions src/browserExt/background.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,13 +27,6 @@ if (!Zotero.isManifestV3) {
browser.action = browser.browserAction;
}

// Some websites break when injecting translation sandbox iframe into
// the website's first iframe (which we do for websites like ebscohost
// where often the translatable site is not the top frame, but first child frame).
const INJECTION_BLACKLIST = new Set([
'lastpass.com'
])

Zotero.Connector_Browser = new function() {
var _tabInfo = {};
var _incompatibleVersionMessageShown;
Expand Down Expand Up @@ -265,10 +258,6 @@ Zotero.Connector_Browser = new function() {
// Injected via the manifest file
return;
} else {
if (Zotero.isManifestV3) {
const host = new URL(url).host;
if (INJECTION_BLACKLIST.has(host)) return;
}
let tabInfo = this.getTabInfo(tab.id);
if (!tabInfo.frameChecked) {
// Also in the first frame detected
Expand Down

0 comments on commit f0d454d

Please sign in to comment.