Do not invoke the App Check token listener for the same token string. (…

…#5993)

* Do not invoke the App Check token listener for the same token string.

* Fix formatting.

* Address comments.

* Create gold-hounds-begin.md

* Use fewer bytes for log messages.

.changeset/gold-hounds-begin.md

@@ -0,0 +1,5 @@
+---
+"@firebase/firestore": patch
+---
+
+Fixed a bug that caused Firestore streams to get restarted with the same App Check token.

packages/firestore/src/api/credentials.ts

@@ -461,10 +461,9 @@ export class FirebaseAppCheckTokenProvider
    * we can unregister it.
    */
   private tokenListener!: AppCheckTokenListener;
-
   private forceRefresh = false;
-
   private appCheck: FirebaseAppCheckInternal | null = null;
+  private latestAppCheckToken: string | null = null;
 
   constructor(
     private appCheckProvider: Provider<AppCheckInternalComponentName>
@@ -482,7 +481,15 @@ export class FirebaseAppCheckTokenProvider
             `Error getting App Check token; using placeholder token instead. Error: ${tokenResult.error.message}`
           );
         }
-        return changeListener(tokenResult.token);
+        const tokenUpdated = tokenResult.token !== this.latestAppCheckToken;
+        this.latestAppCheckToken = tokenResult.token;
+        logDebug(
+          'FirebaseAppCheckTokenProvider',
+          `Received ${tokenUpdated ? 'new' : 'existing'} token.`
+        );
+        return tokenUpdated
+          ? changeListener(tokenResult.token)
+          : Promise.resolve();
       };
 
     this.tokenListener = (tokenResult: AppCheckTokenResult) => {
@@ -534,6 +541,7 @@ export class FirebaseAppCheckTokenProvider
           typeof tokenResult.token === 'string',
           'Invalid tokenResult returned from getToken():' + tokenResult
         );
+        this.latestAppCheckToken = tokenResult.token;
         return new AppCheckToken(tokenResult.token);
       } else {
         return null;