Skip to content

Commit 4aab5bc

Browse files
authored
Fix Logout Warning (#2594)
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
1 parent 4c37afb commit 4aab5bc

File tree

1 file changed

+23
-21
lines changed

1 file changed

+23
-21
lines changed

portal-ui/src/screens/LogoutPage/LogoutPage.tsx

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
// You should have received a copy of the GNU Affero General Public License
1515
// along with this program. If not, see <http://www.gnu.org/licenses/>.
1616

17-
import React from "react";
17+
import React, { useEffect } from "react";
1818
import { useNavigate } from "react-router-dom";
1919
import { useAppDispatch } from "../../store";
2020
import { ErrorResponseHandler } from "../../common/types";
@@ -27,27 +27,29 @@ import LoadingComponent from "../../common/LoadingComponent";
2727
const LogoutPage = () => {
2828
const dispatch = useAppDispatch();
2929
const navigate = useNavigate();
30-
const logout = () => {
31-
const deleteSession = () => {
32-
clearSession();
33-
dispatch(userLogged(false));
34-
localStorage.setItem("userLoggedIn", "");
35-
localStorage.setItem("redirect-path", "");
36-
dispatch(resetSession());
37-
navigate(`/login`);
30+
useEffect(() => {
31+
const logout = () => {
32+
const deleteSession = () => {
33+
clearSession();
34+
dispatch(userLogged(false));
35+
localStorage.setItem("userLoggedIn", "");
36+
localStorage.setItem("redirect-path", "");
37+
dispatch(resetSession());
38+
navigate(`/login`);
39+
};
40+
const state = localStorage.getItem("auth-state");
41+
api
42+
.invoke("POST", `/api/v1/logout`, { state })
43+
.then(() => {
44+
deleteSession();
45+
})
46+
.catch((err: ErrorResponseHandler) => {
47+
console.log(err);
48+
deleteSession();
49+
});
3850
};
39-
const state = localStorage.getItem("auth-state");
40-
api
41-
.invoke("POST", `/api/v1/logout`, { state })
42-
.then(() => {
43-
deleteSession();
44-
})
45-
.catch((err: ErrorResponseHandler) => {
46-
console.log(err);
47-
deleteSession();
48-
});
49-
};
50-
logout();
51+
logout();
52+
}, [dispatch, navigate]);
5153
return <LoadingComponent />;
5254
};
5355

0 commit comments

Comments
 (0)