Skip to content

Bug fix in account linking flows - #8196

Merged
rosalyntan merged 5 commits into
masterfrom
rosalyntan.link
Jun 14, 2021
Merged

Bug fix in account linking flows#8196
rosalyntan merged 5 commits into
masterfrom
rosalyntan.link

Conversation

@rosalyntan

@rosalyntan rosalyntan commented Jun 4, 2021

Copy link
Copy Markdown
Member

Bug fix for an issue where after linking an account with email link, Game Center, or phone auth, the ID token still shows the previous provider (i.e. anonymous), even after force refresh.

[Googler-only internal tracking bug: b/189258684]

@google-oss-bot

google-oss-bot commented Jun 4, 2021

Copy link
Copy Markdown
Collaborator

Coverage Report

Affected SDKs

  • FirebaseAuth-iOS-FirebaseAuth.framework

    SDK overall coverage changed from 68.79% (5aaf38c) to 68.69% (1fc215a) by -0.10%.

    Filename Base (5aaf38c) Head (1fc215a) Diff
    FIRUser.m 69.33% 68.88% -0.45%

Test Logs

@rosalyntan rosalyntan changed the title Bug fix in link with email link flow Bug fix in account linking flows Jun 5, 2021
@rosalyntan
rosalyntan marked this pull request as ready for review June 5, 2021 02:53
@rosalyntan
rosalyntan requested a review from paulb777 June 5, 2021 02:57

@paulb777 paulb777 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does this fix relate to #6521

Comment thread FirebaseAuth/CHANGELOG.md Outdated
@rosalyntan

Copy link
Copy Markdown
Member Author

Does this fix relate to #6521

No, I don't believe the issues are related -- in the other issue, the problem is that the ID token and the approximate expiration time are getting out of sync. In this issue, neither the ID token nor the approximate expiration time were being updated properly, and only in account linking flows.

Comment thread FirebaseAuth/Sources/User/FIRUser.m
Comment thread FirebaseAuth/Sources/User/FIRUser.m
@rosalyntan
rosalyntan merged commit cb0bde5 into master Jun 14, 2021
@rosalyntan
rosalyntan deleted the rosalyntan.link branch June 14, 2021 22:29
@firebase firebase locked and limited conversation to collaborators Jul 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants