Skip to content

Conversation

@mikelehen
Copy link
Contributor

[Port of https://github.com/firebase/firebase-js-sdk/pull/1120]

Fixes #1499.

This reworks our "user listener" into a "credential change listener" that
fires on any token change, even if the User did not change. This allows
us to restart our streams (but not switch mutation queues, etc.) on token
changes.

[Port of firebase/firebase-js-sdk#1120]

Fixes #1499.

This reworks our "user listener" into a "credential change listener" that
fires on any token change, even if the User did not change. This allows
us to restart our streams (but not switch mutation queues, etc.) on token
changes.
Copy link
Contributor

@wilhuff wilhuff left a comment

Choose a reason for hiding this comment

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

LGTM.

Way to follow the three mutually incompatible style guides in effect covering this code!

@mikelehen mikelehen merged commit 186591d into master Aug 15, 2018
@mikelehen mikelehen deleted the mikelehen/handle-all-token-changes branch August 15, 2018 00:22
@firebase firebase locked and limited conversation to collaborators Oct 28, 2019
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.

Firestore - token doesn't update in the Firestore Rules

3 participants