Reject promise in checkError with logoutUser: false and without redir…

…ectTo causes an error #10172
marmelab · Sep 1, 2024 · 05dc2a6 · 05dc2a6
1 parent 733e7a6
commit 05dc2a6
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/packages/ra-core/src/auth/useLogoutIfAccessDenied.ts b/packages/ra-core/src/auth/useLogoutIfAccessDenied.ts
@@ -103,12 +103,17 @@ const useLogoutIfAccessDenied = (): LogoutIfAccessDenied => {
                     if (logoutUser) {
                         logout({}, redirectTo);
                     } else {
-                        if (redirectTo.startsWith('http')) {
-                            // absolute link (e.g. https://my.oidc.server/login)
-                            window.location.href = redirectTo;
+                        if (redirectTo != null) {
+                            if (redirectTo.startsWith('http')) {
+                                // absolute link (e.g. https://my.oidc.server/login)
+                                window.location.href = redirectTo;
+                            } else {
+                                // internal location
+                                navigate(redirectTo);
+                            }
                         } else {
-                            // internal location
-                            navigate(redirectTo);
+                            // If there is not redirect return false saying loguout is not successful
+                            return false;
                         }
                     }