task/TUP-516: Prevent users from getting locked into SMS pairing (#243)

* Prevent users from getting locked into SMS pairing * formatting --------- Co-authored-by: Jake Rosenberg <[email protected]>
TACC · Jun 12, 2023 · 59214c5 · 59214c5
1 parent 94d9b5e
commit 59214c5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 13 deletions.
diff --git a/libs/tup-components/src/mfa/MfaSelection.tsx b/libs/tup-components/src/mfa/MfaSelection.tsx
@@ -5,17 +5,6 @@ import { Link, Navigate } from 'react-router-dom';
 import { useMfa } from '@tacc/tup-hooks';
 
 const MfaSelector: React.FC = () => {
-  const { data } = useMfa();
-
-  // If users have already generated a code with one method,
-  // they will need to complete registration with that method.
-  if (data?.token && data?.token.tokentype === 'sms') {
-    return <Navigate to="/mfa/sms" replace={true} />;
-  }
-  if (data?.token && data?.token.tokentype === 'totp') {
-    return <Navigate to="/mfa/totp" replace={true} />;
-  }
-
   return (
     <div className={styles['mfa-type-container']}>
       Select whether you want to use an MFA token app or SMS texting for

diff --git a/libs/tup-components/src/mfa/MfaValidationPanel.tsx b/libs/tup-components/src/mfa/MfaValidationPanel.tsx
@@ -14,8 +14,8 @@ const MfaValidationPanel: React.FC<{ tokenType: 'totp' | 'sms' }> = ({
   };
 
   const pairingMessage = {
-    sms: '2. Enter the token shown in the app to continue the pairing.',
-    totp: '2. Enter the token sent to your phone number.',
+    totp: '2. Enter the token shown in the app to continue the pairing.',
+    sms: '2. Enter the token sent to your phone number.',
   };
 
   return (