From 33c107dea06ff53ebdae89eb2c7a8e3d26868960 Mon Sep 17 00:00:00 2001
From: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com>
Date: Tue, 28 Jan 2025 09:33:26 +0530
Subject: [PATCH] chore(ui): logout user on unsuccessful refresh attempt
 (#19542)

---
 .../ui/src/components/Auth/AuthProviders/AuthProvider.tsx  | 4 ++++
 openmetadata-ui/src/main/resources/ui/src/rest/LoginAPI.ts | 7 ++++++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx b/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx
index 7ed56792f83..67f2c38587f 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/Auth/AuthProviders/AuthProvider.tsx
@@ -548,6 +548,10 @@ export const AuthProvider = ({
                     resetUserDetails(true);
                   }
                 });
+              }).catch((err) => {
+                resetUserDetails(true);
+
+                return Promise.reject(err);
               });
             } else {
               // If refresh is in progress, queue the request
diff --git a/openmetadata-ui/src/main/resources/ui/src/rest/LoginAPI.ts b/openmetadata-ui/src/main/resources/ui/src/rest/LoginAPI.ts
index 206a12351aa..862ec7c88c2 100644
--- a/openmetadata-ui/src/main/resources/ui/src/rest/LoginAPI.ts
+++ b/openmetadata-ui/src/main/resources/ui/src/rest/LoginAPI.ts
@@ -23,7 +23,12 @@ interface RenewTokenResponse {
 }
 
 export const renewToken = async () => {
-  const data = await axiosClient.get<RenewTokenResponse>(`${BASE_URL}/refresh`);
+  const data = await axiosClient.get<RenewTokenResponse>(
+    `${BASE_URL}/refresh`,
+    // Need to invalidate other status codes
+    // which help is user to force logout
+    { validateStatus: (status) => status === 200 }
+  );
 
   return data.data;
 };