dashboard: fix missing avatar and logout menu (#3819)

pomerium · Dec 17, 2022 · 191eece · 191eece
1 parent 539fd51
commit 191eece
Showing 1 changed file with 31 additions and 29 deletions.
diff --git a/ui/src/components/Header.tsx b/ui/src/components/Header.tsx
@@ -1,8 +1,3 @@
-import { PageData } from "../types";
-import { Avatar } from "./Avatar";
-import Logo from "./Logo";
-import { ToolbarOffset } from "./ToolbarOffset";
-import UserSidebarContent from "./UserSidebarContent";
 import {
   Drawer,
   IconButton,
@@ -18,7 +13,13 @@ import styled from "@mui/material/styles/styled";
 import { get } from "lodash";
 import React, { FC, useState } from "react";
 import { ChevronLeft, ChevronRight, Menu as MenuIcon } from "react-feather";
+
 import LogoURL from "../static/logo_white.svg";
+import { PageData } from "../types";
+import { Avatar } from "./Avatar";
+import Logo from "./Logo";
+import { ToolbarOffset } from "./ToolbarOffset";
+import UserSidebarContent from "./UserSidebarContent";
 
 const DrawerHeader = styled("div")(({ theme }) => ({
   display: "flex",
@@ -47,7 +48,15 @@ const Header: FC<HeaderProps> = ({ includeSidebar, data }) => {
     setAnchorEl(null);
   };
   const userName =
-    get(data, "user.name") || get(data, "user.claims.given_name");
+    get(data, "user.name") ||
+    get(data, "user.claims.given_name") ||
+    get(data, "profile.claims.name") ||
+    get(data, "profile.claims.given_name") ||
+    "";
+  const userPictureUrl =
+    get(data, "user.claims.picture") ||
+    get(data, "profile.claims.picture") ||
+    null;
 
   const handleDrawerOpen = () => {
     setDrawerOpen(true);
@@ -109,32 +118,25 @@ const Header: FC<HeaderProps> = ({ includeSidebar, data }) => {
           </>
         ) : (
           <a href="/.pomerium">
-            <Logo src={ data?.logoUrl || LogoURL }/>
+            <Logo src={data?.logoUrl || LogoURL} />
           </a>
         )}
         <Box flexGrow={1} />
-        {userName && (
-          <>
-            <IconButton color="inherit" onClick={handleMenuOpen}>
-              <Avatar
-                name={userName}
-                url={get(data, "user.claims.picture", null)}
-              />
-            </IconButton>
-            <Menu
-              onClose={handleMenuClose}
-              anchorOrigin={{
-                vertical: "bottom",
-                horizontal: "center",
-              }}
-              keepMounted
-              open={!!anchorEl}
-              anchorEl={anchorEl}
-            >
-              <MenuItem onClick={handleLogout}>Logout</MenuItem>
-            </Menu>
-          </>
-        )}
+        <IconButton color="inherit" onClick={handleMenuOpen}>
+          <Avatar name={userName} url={userPictureUrl} />
+        </IconButton>
+        <Menu
+          onClose={handleMenuClose}
+          anchorOrigin={{
+            vertical: "bottom",
+            horizontal: "center",
+          }}
+          keepMounted
+          open={!!anchorEl}
+          anchorEl={anchorEl}
+        >
+          <MenuItem onClick={handleLogout}>Logout</MenuItem>
+        </Menu>
       </Toolbar>
     </AppBar>
   );