Merge pull request #346 from cstenglein/implement-suggestions

implement suggestions from escapedcat
raspiblitz · Jun 8, 2022 · fdd4950 · fdd4950
2 parents 57077d6 + 963228b
commit fdd4950
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -74,8 +74,8 @@ const App: FC = () => {
             refresh(REFRESH_TIME(payload.expires));
           }
         })
-        .catch(() => {
-          // do nothing
+        .catch((e) => {
+          console.error("Error: token refresh failed with: ", e);
         });
     }
 

diff --git a/src/components/Shared/ConfirmModal/ConfirmModal.tsx b/src/components/Shared/ConfirmModal/ConfirmModal.tsx
@@ -9,7 +9,7 @@ import { MODAL_ROOT } from "../../../util/util";
 
 export type Props = {
   confirmText: string;
-  confirmEndpoint?: string; // TODO remove
+  confirmEndpoint?: string; // TODO #345 remove
   onConfirm?: () => void;
   onClose: () => void;
 };

diff --git a/src/util/util.ts b/src/util/util.ts
@@ -63,7 +63,9 @@ export function checkPropsUndefined(props: object): boolean {
 }
 
 export function parseJwt(token: string): TokenPayload {
-  return JSON.parse(atob(token.split(".")[1]));
+  // 1st part is header, 2nd part is payload, 3rd is signature
+  const payload = token.split(".")[1];
+  return JSON.parse(atob(payload));
 }
 
 export function enableGutter(): void {