Merge pull request #151 from dekart-xyz/fix-auth-error-infinite-loop

Fix auth error infinite loop
dekart-xyz · Dec 6, 2023 · 5b2d9a7 · 5b2d9a7
2 parents 95f97b6 + 1a61905
commit 5b2d9a7
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/src/client/App.js b/src/client/App.js
@@ -61,6 +61,7 @@ function AppRedirect () {
   }, [status, doNotAuthenticate])
 
   if (status === 401 && doNotAuthenticate === false) {
+    // redirect to authentication endpoint from useEffect
     return null
   }
 

diff --git a/src/client/actions/message.js b/src/client/actions/message.js
@@ -51,8 +51,8 @@ export function setError (err, transitive = true) {
   }
 }
 
-export function setHttpError (status, message = '', doNotAuthenticate = false) {
-  return { type: setHttpError.name, status, message, doNotAuthenticate }
+export function setHttpError (status, message = '') {
+  return { type: setHttpError.name, status, message}
 }
 
 export function setStreamError (code, msg) {

diff --git a/src/client/reducers/rootReducer.js b/src/client/reducers/rootReducer.js
@@ -288,12 +288,18 @@ function env (state = defaultEnv, action) {
 
 function httpError (state = {}, action) {
   switch (action.type) {
-    case setHttpError.name:
-      return {
-        status: action.status,
-        message: action.message,
-        doNotAuthenticate: action.doNotAuthenticate
+    case setHttpError.name: {
+      if (action.status === 401 && state.doNotAuthenticate) {
+        // just keep showing auth error, do not override it with other 401
+        return state
+      } else {
+        return {
+          status: action.status,
+          message: action.message,
+          doNotAuthenticate: false
+        }
       }
+    }
     case setRedirectState.name: {
       const err = action.redirectState.getError()
       if (err) {