You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The alert message implies that the function configure_mfa_with_form clears existing mfa methods. However, it appends the new mfa method to the existing ones, since the function add_mfa_method is not called with clear_existing=True
It should say something like "You have successfully added a two-factor authentication method".
Or, more flexibly, it should have the option clear_existing which will then affect the append/clear and also the alert message.
`
def configure_mfa_with_form(allow_cancel=False):
error = None
while True:
mfa_method, password = _configure_mfa(None, error, True, allow_cancel, "Save")
if mfa_method:
try:
add_mfa_method(password, mfa_method)
alert("Your two-factor authentication configuration has been reset.")
return True
except AuthenticationFailed as e:
error = e.args[0]
except Exception as e:
error = str(e)
else:
return None
`
Happy to do a PR if you like :)
The text was updated successfully, but these errors were encountered:
The alert message implies that the function configure_mfa_with_form clears existing mfa methods. However, it appends the new mfa method to the existing ones, since the function add_mfa_method is not called with
clear_existing=True
It should say something like "You have successfully added a two-factor authentication method".
Or, more flexibly, it should have the option
clear_existing
which will then affect the append/clear and also the alert message.`
`
Happy to do a PR if you like :)
The text was updated successfully, but these errors were encountered: