Address comments.

gravitational · Jan 2, 2025 · e77d074 · e77d074
1 parent 6a6d847
commit e77d074
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 22 deletions.
diff --git a/web/packages/teleport/src/JoinTokens/JoinTokens.tsx b/web/packages/teleport/src/JoinTokens/JoinTokens.tsx
@@ -75,9 +75,7 @@ export const JoinTokens = () => {
   const [editingToken, setEditingToken] = useState<JoinToken | null>(null);
   const [tokenToDelete, setTokenToDelete] = useState<JoinToken | null>(null);
   const [joinTokensAttempt, runJoinTokensAttempt, setJoinTokensAttempt] =
-    useAsync(async () => {
-      return ctx.joinTokenService.fetchJoinTokens(null);
-    });
+    useAsync(() => ctx.joinTokenService.fetchJoinTokens(null));
 
   const resources = useResources(
     joinTokensAttempt.data?.items.map(makeTokenResource) || [],

diff --git a/web/packages/teleport/src/MFAContext/MFAContext.tsx b/web/packages/teleport/src/MFAContext/MFAContext.tsx
@@ -1,4 +1,5 @@
-import { PropsWithChildren, createContext, useCallback, useRef } from 'react';
+import { createContext, PropsWithChildren, useCallback, useState } from 'react';
+
 import AuthnDialog from 'teleport/components/AuthnDialog';
 import { useMfa } from 'teleport/lib/useMfa';
 import auth, { MfaChallengeScope } from 'teleport/services/auth/auth';
@@ -11,30 +12,35 @@ export interface MfaContextValue {
 export const MfaContext = createContext<MfaContextValue>(null);
 
 export const MfaContextProvider = ({ children }: PropsWithChildren) => {
-  const allowReuse = useRef(false);
-  const adminMfa = useMfa({
-    req: {
-      scope: MfaChallengeScope.ADMIN_ACTION,
-      allowReuse: allowReuse.current,
-      isMfaRequiredRequest: {
-        admin_action: {},
-      },
-    },
-  });
+  const adminMfa = useMfa({});
 
   const getAdminActionMfaResponse = useCallback(
     async (reusable: boolean = false) => {
-      allowReuse.current = reusable;
-      return (await adminMfa.getChallengeResponse()) || {}; // return an empty challenge to prevent mfa retry.
+      const chal = await auth.getMfaChallenge({
+        scope: MfaChallengeScope.ADMIN_ACTION,
+        allowReuse: reusable,
+        isMfaRequiredRequest: {
+          admin_action: {},
+        },
+      });
+
+      const res = await adminMfa.getChallengeResponse(chal);
+      if (!res) {
+        return {}; // return an empty challenge to prevent mfa retry.
+      }
+
+      return res;
     },
-    [adminMfa, allowReuse]
+    [adminMfa]
   );
 
-  const mfaCtx = {
-    getAdminActionMfaResponse,
-    useMfa,
-  };
-  auth.setMfaContext(mfaCtx);
+  const [mfaCtx, setMfaCtx] = useState<MfaContextValue>();
+
+  if (!mfaCtx) {
+    const mfaCtx = { getAdminActionMfaResponse };
+    setMfaCtx(mfaCtx);
+    auth.setMfaContext(mfaCtx);
+  }
 
   return (
     <MfaContext.Provider value={mfaCtx}>