Don't remove our authentication cookies on an upgrade or downgrade of…

… the add-on, only on uninstall/disable
mozilla-services · Oct 5, 2016 · cf091c5 · cf091c5
1 parent 105d442
commit cf091c5
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/addon/lib/main.js b/addon/lib/main.js
@@ -177,9 +177,11 @@ exports.onUnload = function (reason) {
       deviceInfo: JSON.stringify(deviceInfo)
     }
   });
-  // We can't rely on the server removing cookies, because the request may not
-  // complete locally, since the addon is being destroyed
-  removeCookies();
+  if (reason == "disable" || reason == "uninstall") {
+    // We can't rely on the server removing cookies, because the request may not
+    // complete locally, since the addon is being destroyed
+    removeCookies();
+  }
 };
 
 /** Remove the user and user.sig cookies for the backend */