[firebase_ui_auth] - exception is only throw one time when using startMFAVerification
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- dart, firebase, flutter
- Domain
- authentication, web-dev
Research direction
Start at the firebase_ui_auth implementation of startMFAVerification and compare its handling of the shown provider.sendVerificationCode call after an invalid code. Reproduce the flow with AuthStateChangeAction and verify that repeated validation errors reach the surrounding FirebaseException catch, not only the first attempt.
Written by the indexing model from the issue text.
Description
Is there an existing issue for this?
- I have searched the existing issues and found no duplicates.
What plugin is this bug for?
Firebase UI Auth
What platform(s) does this bug affect?
Web
List of dependencies used.
flutter pub deps -s list
Steps to reproduce
if you have this in your actions:
AuthStateChangeAction<MFARequired>((context, state) async {
try {
await startMFAVerification(
resolver: state.resolver,
context: context,
);
} on FirebaseException catch (e) {
if (!context.mounted) return;
ScaffoldMessenger.of(context).hideCurrentSnackBar();
ScaffoldMessenger.of(context).showSnackBar(
SnackBar(
content: ErrorText(exception: e),
),
);
}
}),
The Exception only only catch the first time as you can see it:
If you press a second time on the validation button
The Exception is not raised to the previous catch statement....but print to the console
Expected Behavior
The the error as many time needed.
One you have an error, In was expecting form this helper method to restart:
provider.sendVerificationCode(
hint: hint as fba.PhoneMultiFactorInfo,
multiFactorSession: session,
action: AuthAction.none,
);
Otherwise once you have an error, you need to go to the previous page and type correctly the pin.
I'm expecting user to sometime do mistakes when entering the code, so this can be very annoying from an end user perspective.
Actual Behavior
Only trigger one time
Additional Information
No response
- Dominant language
- Dart
- Stars
- 152
- Forks
- 141
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 9
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from firebase/FirebaseUI-Flutter
-
auth blocked: customer response enhancement Keep Open
Difficulty 3/5 1-2 days Newbie friendliness 35/100
firebase/FirebaseUI-Flutter#418 · 7 comments · 3 reactions ·
-
auth Keep Open Needs Attention
Difficulty 4/5 3-5 days Newbie friendliness 20/100
firebase/FirebaseUI-Flutter#351 · 3 comments · 1 reaction ·
-
enhancement Keep Open storage
Difficulty 4/5 3-5 days Newbie friendliness 38/100
firebase/FirebaseUI-Flutter#290 · 2 comments ·
-
auth Keep Open platform: android
Difficulty 4/5 3-5 days Newbie friendliness 25/100
firebase/FirebaseUI-Flutter#283 · 9 comments · 1 reaction ·
-
auth bug Keep Open Needs Attention
Difficulty 4/5 3-5 days Newbie friendliness 25/100
firebase/FirebaseUI-Flutter#261 · 6 comments · 3 reactions ·
All issues in firebase/FirebaseUI-Flutter
Similar issues
-
status: needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
a2ui-project/a2ui#2782 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
MaikuB/flutter_local_notifications#2832 · 1 comment ·
-
bug triage
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
fluttercommunity/plus_plugins#3979 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
from.page-issue
Difficulty 1/5 Under an hour Newbie friendliness 85/100