From 0b1f837c2ad2f3b4c1018c8b8cdbbefe0ee87ffe Mon Sep 17 00:00:00 2001 From: anovazzi1 Date: Thu, 5 Oct 2023 18:04:21 -0300 Subject: [PATCH] fix(api.tsx): remove unnecessary try-catch block and add error logging for better debugging fix(index.ts): remove console.log statement for error logging --- src/frontend/src/controllers/API/api.tsx | 14 ++++++++------ src/frontend/src/controllers/API/index.ts | 1 - 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/frontend/src/controllers/API/api.tsx b/src/frontend/src/controllers/API/api.tsx index 75be74fac..2847fe8b0 100644 --- a/src/frontend/src/controllers/API/api.tsx +++ b/src/frontend/src/controllers/API/api.tsx @@ -31,13 +31,11 @@ function ApiInterceptor() { logout(); navigate("/login"); } - - const res = await renewAccessToken(refreshToken); - if (res?.data?.access_token && res?.data?.refresh_token) { - login(res?.data?.access_token, res?.data?.refresh_token); - } - try { + const res = await renewAccessToken(refreshToken); + if (res?.data?.access_token && res?.data?.refresh_token) { + login(res?.data?.access_token, res?.data?.refresh_token); + } if (error?.config?.headers) { delete error.config.headers["Authorization"]; error.config.headers["Authorization"] = `Bearer ${cookies.get( @@ -50,6 +48,10 @@ function ApiInterceptor() { if (axios.isAxiosError(error) && error.response?.status === 401) { logout(); navigate("/login"); + } else { + console.error(error); + logout(); + navigate("/login"); } } } diff --git a/src/frontend/src/controllers/API/index.ts b/src/frontend/src/controllers/API/index.ts index ddc8509b7..f66233afd 100644 --- a/src/frontend/src/controllers/API/index.ts +++ b/src/frontend/src/controllers/API/index.ts @@ -399,7 +399,6 @@ export async function renewAccessToken(token: string) { return await api.post(`${BASE_URL_API}refresh?token=${token}`); } } catch (error) { - console.log("Error:", error); throw error; } }