- πΊπΈUnited States cmlara
Needs rebase to 2.x
Also the message "You do not have permission to setup TFA. Please contact administrator" would likely be better along the lines of:
"You are required to setup two-factor authentication however your account does not have the necessary permissions. Please contact an administrator. You have @remaining attempt(s) left. After this you will be unable to login." Run through formatPlural() so that users know the risk of not taking action.
- Status changed to Needs review
about 1 year ago 6:14pm 10 September 2023 - last update
about 1 year ago 21 pass - last update
about 1 year ago Composer require failure - πΊπΈUnited States cmlara
Critical change: the hasSkipped() needs to be outside the if/else check as it applies to both those user with and without permission to setup their TFA tokens.
- last update
about 1 year ago 21 pass - last update
about 1 year ago 21 pass - πΊπΈUnited States cmlara
Here is the 1.x patch
The change to TfaPasswordResetTest.php is due to π TfaPasswordResetTest does not directly test TFA prompts. Active testing for the messages instead of validating the login forms.
-
cmlara β
committed a8ff1870 on 2.x
Issue #3089931 by vatsalkhanna, cmlara: Users are directed to TFA...
-
cmlara β
committed a8ff1870 on 2.x
-
cmlara β
committed a5edf54d on 8.x-1.x
Issue #3089931 by vatsalkhanna, cmlara: Users are directed to TFA...
-
cmlara β
committed a5edf54d on 8.x-1.x
- Status changed to Fixed
about 1 year ago 5:16am 16 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.