From 8a95089a484999c7451314f4193e1c5e099bcac0 Mon Sep 17 00:00:00 2001
From: Manav Rathi <manav@mrmr.io>
Date: Thu, 21 Nov 2024 11:44:41 +0530
Subject: [PATCH] Swap

---
 web/packages/accounts/services/logout.ts |  4 ++--
 web/packages/accounts/services/user.ts   | 26 ++----------------------
 2 files changed, 4 insertions(+), 26 deletions(-)

diff --git a/web/packages/accounts/services/logout.ts b/web/packages/accounts/services/logout.ts
index 720e51e59c..c88c26eece 100644
--- a/web/packages/accounts/services/logout.ts
+++ b/web/packages/accounts/services/logout.ts
@@ -5,7 +5,7 @@ import log from "@/base/log";
 import localForage from "@ente/shared/storage/localForage";
 import { clearKeys } from "@ente/shared/storage/sessionStorage";
 import { clearStashedRedirect } from "./redirect";
-import { logout as remoteLogout } from "./user";
+import { remoteLogoutIfNeeded } from "./user";
 
 /**
  * Logout sequence common to all apps that rely on the accounts package.
@@ -24,7 +24,7 @@ export const accountLogout = async () => {
     log.info("logout (account)");
 
     try {
-        await remoteLogout();
+        await remoteLogoutIfNeeded();
     } catch (e) {
         ignoreError("Remote", e);
     }
diff --git a/web/packages/accounts/services/user.ts b/web/packages/accounts/services/user.ts
index 7afe88f19d..84f7eca7fb 100644
--- a/web/packages/accounts/services/user.ts
+++ b/web/packages/accounts/services/user.ts
@@ -100,35 +100,13 @@ export const remoteLogoutIfNeeded = async () => {
         method: "POST",
         headers,
     });
-    if (!res.ok && res.status == 401) {
+    if (res.status == 401) {
         // Ignore if we get a 401 Unauthorized, this is expected to happen on
         // token expiry.
         return;
     }
-    ensureOk(res);
-};
 
-export const logout = async () => {
-    try {
-        const token = getToken();
-        await HTTPService.post(await apiURL("/users/logout"), null, undefined, {
-            "X-Auth-Token": token,
-        });
-    } catch (e) {
-        // ignore if token missing can be triggered during sign up.
-        if (e instanceof Error && e.message === CustomError.TOKEN_MISSING) {
-            return;
-        }
-        // ignore if unauthorized, can be triggered during on token expiry.
-        else if (
-            e instanceof ApiError &&
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
-            e.httpStatusCode === HttpStatusCode.Unauthorized
-        ) {
-            return;
-        }
-        throw e;
-    }
+    ensureOk(res);
 };
 
 export const verifyTwoFactor = async (code: string, sessionID: string) => {