Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix(auth): use correct app instance (vs always default) in multifactor and phone auth #7564

Merged
merged 7 commits into from
Jan 30, 2024

Conversation

Somebodybro
Copy link
Contributor

@Somebodybro Somebodybro commented Jan 12, 2024

Description

There is was/is a bug in the firebase sdk(12th of january 2024) that prevents multifactor login from being completed with a secondary app instance, but after that is updated, secondary instances should work with mfa

Related issues

Release Summary

Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
    • Yes
  • My change supports the following platforms;
    • Android
    • iOS
  • My change includes tests;
    • e2e tests added or updated in packages/\*\*/e2e
    • jest tests added or updated in packages/\*\*/__tests__
  • I have updated TypeScript types that are affected by my change.
  • This is a breaking change;
    • Yes
    • No

Test Plan


Think react-native-firebase is great? Please consider supporting the project with any of the below:

🔥

@CLAassistant
Copy link

CLAassistant commented Jan 12, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

vercel bot commented Jan 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-native-firebase ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 25, 2024 6:02am
react-native-firebase-next ❌ Failed (Inspect) Jan 25, 2024 6:02am

Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @Somebodybro 👋 - thanks for working through this both here and upstream! Well done

Upstream note: the PR you proposed to firebase-ios-sdk was merged and it was just tagged for firebase-ios-sdk 10.21.0 so hopefully that's out soon, but it is not out yet as I type this

I control the sdk versions via separate PRs, but I don't think we need to wait on this? It's still correct here I think, it just won't completely work until upstream 10.21.0 is out?

The only thing remaining here is for you to run lint:ios:fix and commit the results - I did that locally but I can't push the result out as your repo main branch rejects the push, so you'll have to do it yourself I think?

@mikehardy mikehardy added Workflow: Waiting for User Response Blocked waiting for user response. plugin: authentication Firebase Authentication labels Jan 23, 2024
@mikehardy mikehardy changed the title fix: fixed multifactor and phone auth always using default instance fix(auth): use correct app instance (vs always default) in multifactor and phone auth Jan 23, 2024
@Somebodybro Somebodybro requested a review from mikehardy January 25, 2024 05:59
@Somebodybro
Copy link
Contributor Author

Somebodybro commented Jan 25, 2024

Hei! I ran lint:ios:fix and the repo should be linted.
Can't believe I completely forgot to do that.

Sorry I was not working on this for a couple of days. It was nice working with this library (and upstream).

@mikehardy mikehardy added Workflow: Pending Merge Waiting on CI or similar and removed Workflow: Waiting for User Response Blocked waiting for user response. labels Jan 30, 2024
Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fantastic

@mikehardy mikehardy merged commit ff32fd3 into invertase:main Jan 30, 2024
17 checks passed
@mikehardy mikehardy removed the Workflow: Pending Merge Waiting on CI or similar label Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin: authentication Firebase Authentication
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple firebase projects with multi factor authentication
3 participants