Skip to content

fix(auth, android): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session #12488

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Mar 15, 2024

Conversation

Lyokone
Copy link
Contributor

@Lyokone Lyokone commented Mar 14, 2024

Description

When signing out we weren't removing the old multifactor objects.

Related Issues

closes #12486

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
  • All existing and new tests are passing.
  • I updated/added relevant documentation (doc comments with ///).
  • The analyzer (melos run analyze) does not report any problems on my PR.
  • I read and followed the Flutter Style Guide.
  • I signed the CLA.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change.
  • No, this is not a breaking change.

…urn an empty list if signing out in the same app session
@Lyokone Lyokone changed the title fix(auth): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session fix(auth, android): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session Mar 14, 2024
@Lyokone Lyokone changed the title fix(auth, android): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session fix(auth, android): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session Mar 14, 2024
@Lyokone Lyokone merged commit 04280a3 into master Mar 15, 2024
22 checks passed
@Lyokone Lyokone deleted the fix/12486 branch March 15, 2024 10:04
LocLt-Mobile pushed a commit to guide-inc-org/guide-flutter_fire that referenced this pull request Apr 5, 2024
…ors` to return an empty list if signing out in the same app session (firebase#12488)

* fix(auth): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session

* format

* fix tests
LocLt-Mobile pushed a commit to guide-inc-org/guide-flutter_fire that referenced this pull request Apr 5, 2024
…ors` to return an empty list if signing out in the same app session (firebase#12488)

* fix(auth): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session

* format

* fix tests
LocLt-Mobile pushed a commit to guide-inc-org/guide-flutter_fire that referenced this pull request Apr 5, 2024
…ors` to return an empty list if signing out in the same app session (firebase#12488)

* fix(auth): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session

* format

* fix tests
LocLt-Mobile pushed a commit to guide-inc-org/guide-flutter_fire that referenced this pull request Apr 5, 2024
…ors` to return an empty list if signing out in the same app session (firebase#12488)

* fix(auth): fixing an issue that could cause getEnrolledFactors to return an empty list if signing out in the same app session

* format

* fix tests
@firebase firebase locked and limited conversation to collaborators Apr 15, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
3 participants