Skip to content

Commit

Permalink
Don't force allowlisted URLs into Google Container
Browse files Browse the repository at this point in the history
  • Loading branch information
aaronkollasch committed May 23, 2022
1 parent 7675291 commit b724eb6
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion background.js
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,8 @@ function shouldContainInto (url, tab) {
return false;
}

let handleUrl = isGoogleURL(url) || (extensionSettings.allowlist.length!=0 && isAllowlistedURL(url));
let allowlistUrl = (extensionSettings.allowlist.length!=0 && isAllowlistedURL(url));
let handleUrl = isGoogleURL(url) || allowlistUrl;

if (handleUrl && extensionSettings.whitelist.length!=0 && isWhitelistedURL(url)) {
handleUrl = false;
Expand Down Expand Up @@ -380,6 +381,11 @@ function shouldContainInto (url, tab) {
return false;
}

if (allowlistUrl) {
// Don't force an allowlisted URL to be in the Google Container
return false;
}

// Google-URL outside of Google Container Tab
// Should contain into Google Container
return googleCookieStoreId;
Expand Down

0 comments on commit b724eb6

Please sign in to comment.