From 3d5ef06e388d68ad4c2cbfd9d7494840d833d905 Mon Sep 17 00:00:00 2001
From: Martin Hradil <mhradil@redhat.com>
Date: Tue, 10 Oct 2023 15:22:35 +0200
Subject: [PATCH] logout: reload user info after logout, instead of assuming
null (#4383)
because the UI doesn't really know about the anonymous user, except from the API
so.. logout would set user to null, going through the codepath that redirects to /login, which would go to github and log in again
instead, reloading the info when in anonymous mode, the user gets set to the anonymous user, with no redirects
Issue: AAH-2726
(cherry picked from commit ceba5a3f5c9e20c5989cb7898572e028ba8f0f74)
---
CHANGES/2726.misc | 1 +
src/loaders/standalone/layout.tsx | 6 +++++-
2 files changed, 6 insertions(+), 1 deletion(-)
create mode 100644 CHANGES/2726.misc
diff --git a/CHANGES/2726.misc b/CHANGES/2726.misc
new file mode 100644
index 0000000000..390adffaba
--- /dev/null
+++ b/CHANGES/2726.misc
@@ -0,0 +1 @@
+logout: reload user info after logout, instead of assuming null
diff --git a/src/loaders/standalone/layout.tsx b/src/loaders/standalone/layout.tsx
index 66b7853277..6be2bb67dc 100644
--- a/src/loaders/standalone/layout.tsx
+++ b/src/loaders/standalone/layout.tsx
@@ -77,7 +77,11 @@ export const StandaloneLayout = ({
<DropdownItem
key='logout'
aria-label={'logout'}
- onClick={() => ActiveUserAPI.logout().then(() => setUser(null))}
+ onClick={() =>
+ ActiveUserAPI.logout()
+ .then(() => ActiveUserAPI.getUser().catch(() => null))
+ .then((user) => setUser(user))
+ }
>
{t`Logout`}
</DropdownItem>,