Skip to content

Commit

Permalink
Merge pull request #6160 from bithyve/fix/2FA-otp
Browse files Browse the repository at this point in the history
fix: 2FA otp reset
  • Loading branch information
ben-kaufman authored Mar 10, 2025
2 parents f317b7e + 0d8a453 commit 402f2ec
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 5 deletions.
4 changes: 4 additions & 0 deletions src/screens/SigningDevices/SetupSigningServer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -60,13 +60,16 @@ function SetupSigningServer({ route }: { route }) {
if (valid) {
setIsSetupValidated(valid);
showValidationModal(false);
setOtp('');
} else {
showValidationModal(false);
showToast('Invalid OTP. Please try again!');
setOtp('');
}
} catch (err) {
showValidationModal(false);
showToast(`${err.message}`);
setOtp('');
}
};

Expand Down Expand Up @@ -233,6 +236,7 @@ function SetupSigningServer({ route }: { route }) {
visible={validationModal}
close={() => {
showValidationModal(false);
setOtp('');
}}
title={common.confirm2FACodeTitle}
subTitle={common.confirm2FACodeSubtitle}
Expand Down
15 changes: 10 additions & 5 deletions src/screens/Vault/ChoosePolicyNew.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -222,21 +222,25 @@ function ChoosePolicyNew({ navigation, route }) {
setDelayModal(true);
dispatch(setSignerPolicyError('idle'));
showValidationModal(false);
setOtp('');
}, 100);
} else {
// more restrictive policy update - immediate
setTimeout(() => {
setConfigureSuccessModal(true);
dispatch(setSignerPolicyError('idle'));
showValidationModal(false);
setOtp('');
}, 100);
}
} else {
setIsLoading(false);
dispatch(setSignerPolicyError('idle'));
showValidationModal(false);
// resetFields();
showToast('Something went wrong. Please try again');
setTimeout(() => {
setIsLoading(false);
dispatch(setSignerPolicyError('idle'));
showValidationModal(false);
setOtp('');
showToast('Something went wrong. Please try again');
}, 100);
}
}
}, [policyError]);
Expand Down Expand Up @@ -402,6 +406,7 @@ function ChoosePolicyNew({ navigation, route }) {
visible={validationModal}
close={() => {
showValidationModal(false);
setOtp('');
// resetFields();
}}
title={common.confirm2FACodeTitle}
Expand Down
2 changes: 2 additions & 0 deletions src/screens/Vault/HardwareModalMap.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2271,8 +2271,10 @@ function HardwareModalMap({
close={() => {
if (type === SignerType.POLICY_SERVER && mode === InteracationMode.HEALTH_CHECK) {
setSigningServerHealthCheckOTPModal(false);
setOtp('');
} else {
setSigningServerRecoverOTPModal(false);
setOtp('');
}
close();
}}
Expand Down

0 comments on commit 402f2ec

Please sign in to comment.