Added a notification for FPI=true and FF version=58

Cookie-AutoDelete · Jan 23, 2018 · 9e067a9 · 9e067a9
1 parent db54f00
commit 9e067a9
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/background.js b/src/background.js
@@ -320,6 +320,14 @@ const onStartUp = async () => {
 					key: "firstPartyIsolateSetting", value: setting.value
 				}
 			});
+			if (browserVersion === "58" && setting.value) {
+				browser.notifications.create("FPI_NOTIFICATION", {
+					"type": "basic",
+					"iconUrl": browser.extension.getURL("icons/icon_48.png"),
+					"title": "First Party Isolation Detected",
+					"message": "Please turn off privacy.firstparty.isolate and restart the browser as it breaks cookie cleanup"
+				});
+			}
 		}
 	}
 	// Store which browser environment in cache

diff --git a/src/redux/Actions.js b/src/redux/Actions.js
@@ -127,6 +127,16 @@ export const cookieCleanup = (options) => async (dispatch, getState) => {
 			greyCleanup: false, ignoreOpenTabs: false
 		};
 	}
+
+	if (getState().cache.browserVersion === "58" && getState().cache.firstPartyIsolateSetting) {
+		browser.notifications.create("FPI_NOTIFICATION", {
+			"type": "basic",
+			"iconUrl": browser.extension.getURL("icons/icon_48.png"),
+			"title": "First Party Isolation Detected",
+			"message": "Please turn off privacy.firstparty.isolate and restart the browser as it breaks cookie cleanup"
+		});
+	}
+
 	const cleanupDoneObject = await cleanCookiesOperation(getState(), newOptions);
 	const {
 		recentlyCleaned, setOfDeletedDomainCookies