Merge pull request #1157 from zhx828/hotfix-2

Do not reload the page when authentication endpoint failed
oncokb · Sep 25, 2024 · 7b15ec3 · 7b15ec3
2 parents f96b287 + 7a81790
commit 7b15ec3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/main/webapp/app/config/constants.tsx b/src/main/webapp/app/config/constants.tsx
@@ -609,6 +609,10 @@ export enum PAGE_ROUTE {
   ACCOUNT_ACTIVE_TRIAL_FINISH = '/account/active-trial/finish',
 }
 
+export enum API_ROUTE {
+  AUTHENTICATE = '/api/authenticate',
+}
+
 export enum TABLE_COLUMN_KEY {
   HUGO_SYMBOL = 'HUGO_SYMBOL',
   ALTERATION = 'ALTERATION',

diff --git a/src/main/webapp/app/index.tsx b/src/main/webapp/app/index.tsx
@@ -23,7 +23,7 @@ import {
   getPublicWebsiteToken,
   getStoredRecaptchaToken,
 } from 'app/indexUtils';
-import { UNAUTHORIZED_ALLOWED_PATH } from 'app/config/constants';
+import { API_ROUTE, UNAUTHORIZED_ALLOWED_PATH } from 'app/config/constants';
 import { AppConfig, AppProfile } from 'app/appConfig';
 
 assignPublicToken();
@@ -79,6 +79,8 @@ superagent.Request.prototype.end = function (callback) {
       AppConfig.serverConfig.token &&
       AppConfig.serverConfig.appProfile === AppProfile.PROD &&
       response.req &&
+      // we do not reload the page when the /api/authenticate failed
+      response.req.url !== API_ROUTE.AUTHENTICATE &&
       UNAUTHORIZED_ALLOWED_PATH.some(path =>
         window.location.pathname.endsWith(path)
       )