Merge pull request Tencent#264 from qianxinfeng/dev

fix remove cookie fail when it is set path=/ or top domain
sunderls · Aug 17, 2019 · c8addd4 · c8addd4
2 parents dd782b7 + 7ddaeac
commit c8addd4
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/dist/vconsole.min.js b/dist/vconsole.min.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/storage/storage.js b/src/storage/storage.js
@@ -217,10 +217,13 @@ class VConsoleStorageTab extends VConsolePlugin {
     if (!document.cookie || !navigator.cookieEnabled) {
       return;
     }
-
+    let hostname = window.location.hostname;
     let list = this.getCookieList();
     for (var i=0; i<list.length; i++) {
-      document.cookie = list[i].name + '=;expires=Thu, 01 Jan 1970 00:00:00 GMT';
+      let name=list[i].name;
+      document.cookie = `${name}=;expires=Thu, 01 Jan 1970 00:00:00 GMT`;
+      document.cookie = `${name}=;expires=Thu, 01 Jan 1970 00:00:00 GMT;path=/`;
+      document.cookie = `${name}=;expires=Thu, 01 Jan 1970 00:00:00 GMT;path=/;domain=.${hostname.split('.').slice(-2).join('.')}`;
     }
     this.renderStorage();
   }