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

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

Merged
merged 5 commits into from
Feb 12, 2022

Conversation

ehsannas
Copy link
Contributor

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Feb 10, 2022

🦋 Changeset detected

Latest commit: 19d5c9c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@firebase/firestore Patch
firebase Patch
@firebase/firestore-compat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 10, 2022

Size Report 1

Affected Products

  • @firebase/firestore

    TypeBase (3fe2c4a)Merge (1aa9044)Diff
    browser233 kB233 kB+372 B (+0.2%)
    esm5290 kB291 kB+376 B (+0.1%)
    main462 kB463 kB+626 B (+0.1%)
    module233 kB233 kB+372 B (+0.2%)
    react-native233 kB233 kB+372 B (+0.2%)
  • bundle

    TypeBase (3fe2c4a)Merge (1aa9044)Diff
    firestore (Persistence)234 kB234 kB+373 B (+0.2%)
    firestore (Query Cursors)190 kB190 kB+181 B (+0.1%)
    firestore (Query)191 kB191 kB+181 B (+0.1%)
    firestore (Read data once)179 kB179 kB+181 B (+0.1%)
    firestore (Realtime updates)181 kB182 kB+181 B (+0.1%)
    firestore (Transaction)164 kB164 kB+178 B (+0.1%)
    firestore (Write data)163 kB163 kB+178 B (+0.1%)
  • firebase

    TypeBase (3fe2c4a)Merge (1aa9044)Diff
    firebase-compat.js757 kB758 kB+367 B (+0.0%)
    firebase-firestore-compat.js285 kB285 kB+367 B (+0.1%)
    firebase-firestore.js783 kB784 kB+842 B (+0.1%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/vf7ZsbIVJb.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 10, 2022

Size Analysis Report 1

Affected Products

  • @firebase/firestore

    • AbstractUserDataWriter

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size23.4 kB23.6 kB+177 B (+0.8%)
      size-with-ext-deps72.0 kB72.2 kB+178 B (+0.2%)
    • Bytes

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size17.3 kB17.5 kB+177 B (+1.0%)
      size-with-ext-deps65.8 kB65.9 kB+178 B (+0.3%)
    • CACHE_SIZE_UNLIMITED

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • CollectionReference

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size19.4 kB19.6 kB+177 B (+0.9%)
      size-with-ext-deps67.9 kB68.1 kB+178 B (+0.3%)
    • DocumentReference

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size19.4 kB19.6 kB+177 B (+0.9%)
      size-with-ext-deps67.9 kB68.1 kB+178 B (+0.3%)
    • DocumentSnapshot

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size22.9 kB23.1 kB+177 B (+0.8%)
      size-with-ext-deps71.4 kB71.6 kB+178 B (+0.2%)
    • FieldPath

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size18.7 kB18.9 kB+177 B (+0.9%)
      size-with-ext-deps67.2 kB67.4 kB+178 B (+0.3%)
    • FieldValue

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • Firestore

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.6 kB+178 B (+0.3%)
    • FirestoreError

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.6 kB+178 B (+0.3%)
    • GeoPoint

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.6 kB16.7 kB+177 B (+1.1%)
      size-with-ext-deps65.1 kB65.2 kB+178 B (+0.3%)
    • LoadBundleTask

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.9 kB17.0 kB+177 B (+1.1%)
      size-with-ext-deps65.3 kB65.5 kB+178 B (+0.3%)
    • Query

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.1 kB16.3 kB+177 B (+1.1%)
      size-with-ext-deps64.6 kB64.8 kB+178 B (+0.3%)
    • QueryConstraint

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • QueryDocumentSnapshot

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size22.9 kB23.1 kB+177 B (+0.8%)
      size-with-ext-deps71.4 kB71.6 kB+178 B (+0.2%)
    • QuerySnapshot

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size24.8 kB25.0 kB+177 B (+0.7%)
      size-with-ext-deps73.3 kB73.5 kB+178 B (+0.2%)
    • SnapshotMetadata

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.2 kB16.3 kB+177 B (+1.1%)
      size-with-ext-deps64.6 kB64.8 kB+178 B (+0.3%)
    • Timestamp

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size17.2 kB17.4 kB+177 B (+1.0%)
      size-with-ext-deps65.7 kB65.8 kB+178 B (+0.3%)
    • Transaction

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size42.0 kB42.1 kB+177 B (+0.4%)
      size-with-ext-deps90.6 kB90.8 kB+178 B (+0.2%)
    • WriteBatch

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size38.8 kB38.9 kB+177 B (+0.5%)
      size-with-ext-deps87.4 kB87.6 kB+178 B (+0.2%)
    • _DatabaseId

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.6 kB+178 B (+0.3%)
    • _DocumentKey

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size25.1 kB25.3 kB+177 B (+0.7%)
      size-with-ext-deps73.6 kB73.8 kB+178 B (+0.2%)
    • _EmptyAppCheckTokenProvider

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.1 kB16.3 kB+177 B (+1.1%)
      size-with-ext-deps64.6 kB64.8 kB+178 B (+0.3%)
    • _EmptyAuthCredentialsProvider

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • _FieldPath

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size18.4 kB18.6 kB+177 B (+1.0%)
      size-with-ext-deps66.9 kB67.1 kB+178 B (+0.3%)
    • _cast

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.8 kB16.9 kB+177 B (+1.1%)
      size-with-ext-deps65.2 kB65.4 kB+178 B (+0.3%)
    • _debugAssert

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • _isBase64Available

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • _logWarn

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.1 kB16.3 kB+177 B (+1.1%)
      size-with-ext-deps64.6 kB64.7 kB+178 B (+0.3%)
    • _setIndexConfiguration

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size23.9 kB24.1 kB+177 B (+0.7%)
      size-with-ext-deps72.4 kB72.6 kB+178 B (+0.2%)
    • _validateIsNotUsedTogether

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • addDoc

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size106 kB106 kB+177 B (+0.2%)
      size-with-ext-deps156 kB156 kB+178 B (+0.1%)
    • arrayRemove

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size28.2 kB28.4 kB+177 B (+0.6%)
      size-with-ext-deps76.8 kB77.0 kB+178 B (+0.2%)
    • arrayUnion

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size28.2 kB28.4 kB+177 B (+0.6%)
      size-with-ext-deps76.8 kB77.0 kB+178 B (+0.2%)
    • clearIndexedDbPersistence

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size24.2 kB24.6 kB+368 B (+1.5%)
      size-with-ext-deps73.0 kB73.3 kB+370 B (+0.5%)
    • collection

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size20.1 kB20.3 kB+177 B (+0.9%)
      size-with-ext-deps68.7 kB68.9 kB+178 B (+0.3%)
    • collectionGroup

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size19.3 kB19.5 kB+177 B (+0.9%)
      size-with-ext-deps67.8 kB68.0 kB+178 B (+0.3%)
    • connectFirestoreEmulator

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size17.8 kB18.0 kB+177 B (+1.0%)
      size-with-ext-deps70.0 kB70.2 kB+178 B (+0.3%)
    • deleteDoc

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size97.3 kB97.5 kB+177 B (+0.2%)
      size-with-ext-deps147 kB147 kB+178 B (+0.1%)
    • deleteField

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.4 kB16.6 kB+177 B (+1.1%)
      size-with-ext-deps64.9 kB65.0 kB+178 B (+0.3%)
    • disableNetwork

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size86.0 kB86.2 kB+177 B (+0.2%)
      size-with-ext-deps136 kB136 kB+178 B (+0.1%)
    • doc

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size20.2 kB20.4 kB+177 B (+0.9%)
      size-with-ext-deps68.8 kB68.9 kB+178 B (+0.3%)
    • documentId

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size18.7 kB18.9 kB+177 B (+0.9%)
      size-with-ext-deps67.2 kB67.4 kB+178 B (+0.3%)
    • enableIndexedDbPersistence

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size146 kB146 kB+369 B (+0.3%)
      size-with-ext-deps196 kB196 kB+370 B (+0.2%)
    • enableMultiTabIndexedDbPersistence

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size177 kB178 kB+372 B (+0.2%)
      size-with-ext-deps227 kB228 kB+373 B (+0.2%)
    • enableNetwork

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size86.0 kB86.1 kB+177 B (+0.2%)
      size-with-ext-deps136 kB136 kB+178 B (+0.1%)
    • endAt

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size34.1 kB34.3 kB+177 B (+0.5%)
      size-with-ext-deps82.7 kB82.9 kB+178 B (+0.2%)
    • endBefore

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size34.2 kB34.3 kB+177 B (+0.5%)
      size-with-ext-deps82.7 kB82.9 kB+178 B (+0.2%)
    • ensureFirestoreConfigured

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.1 kB16.3 kB+177 B (+1.1%)
      size-with-ext-deps64.6 kB64.8 kB+178 B (+0.3%)
    • executeWrite

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size96.4 kB96.6 kB+177 B (+0.2%)
      size-with-ext-deps146 kB146 kB+178 B (+0.1%)
    • getDoc

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size122 kB122 kB+180 B (+0.1%)
      size-with-ext-deps172 kB172 kB+181 B (+0.1%)
    • getDocFromCache

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size79.1 kB79.3 kB+177 B (+0.2%)
      size-with-ext-deps128 kB128 kB+178 B (+0.1%)
    • getDocFromServer

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size122 kB122 kB+180 B (+0.1%)
      size-with-ext-deps172 kB172 kB+181 B (+0.1%)
    • getDocs

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size124 kB124 kB+180 B (+0.1%)
      size-with-ext-deps174 kB174 kB+181 B (+0.1%)
    • getDocsFromCache

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size86.3 kB86.5 kB+177 B (+0.2%)
      size-with-ext-deps135 kB135 kB+178 B (+0.1%)
    • getDocsFromServer

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size124 kB124 kB+180 B (+0.1%)
      size-with-ext-deps174 kB174 kB+181 B (+0.1%)
    • getFirestore

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.1 kB16.3 kB+177 B (+1.1%)
      size-with-ext-deps64.8 kB64.9 kB+178 B (+0.3%)
    • increment

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.8 kB17.0 kB+177 B (+1.1%)
      size-with-ext-deps65.3 kB65.5 kB+178 B (+0.3%)
    • initializeFirestore

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.6 kB16.8 kB+177 B (+1.1%)
      size-with-ext-deps65.5 kB65.7 kB+178 B (+0.3%)
    • limit

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.8 kB17.0 kB+177 B (+1.1%)
      size-with-ext-deps65.3 kB65.5 kB+178 B (+0.3%)
    • limitToLast

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.9 kB17.0 kB+177 B (+1.0%)
      size-with-ext-deps65.4 kB65.5 kB+178 B (+0.3%)
    • loadBundle

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size94.8 kB94.9 kB+177 B (+0.2%)
      size-with-ext-deps145 kB145 kB+178 B (+0.1%)
    • namedQuery

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size72.6 kB72.8 kB+177 B (+0.2%)
      size-with-ext-deps121 kB121 kB+178 B (+0.1%)
    • onSnapshot

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size124 kB125 kB+180 B (+0.1%)
      size-with-ext-deps174 kB174 kB+181 B (+0.1%)
    • onSnapshotsInSync

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size114 kB114 kB+180 B (+0.2%)
      size-with-ext-deps164 kB164 kB+181 B (+0.1%)
    • orderBy

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size21.2 kB21.3 kB+177 B (+0.8%)
      size-with-ext-deps69.7 kB69.9 kB+178 B (+0.3%)
    • query

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • queryEqual

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size25.6 kB25.8 kB+177 B (+0.7%)
      size-with-ext-deps74.1 kB74.3 kB+178 B (+0.2%)
    • refEqual

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size19.6 kB19.8 kB+177 B (+0.9%)
      size-with-ext-deps68.2 kB68.3 kB+178 B (+0.3%)
    • runTransaction

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size107 kB107 kB+177 B (+0.2%)
      size-with-ext-deps157 kB157 kB+178 B (+0.1%)
    • serverTimestamp

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.3 kB16.5 kB+177 B (+1.1%)
      size-with-ext-deps64.8 kB65.0 kB+178 B (+0.3%)
    • setDoc

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size105 kB106 kB+177 B (+0.2%)
      size-with-ext-deps155 kB155 kB+178 B (+0.1%)
    • setLogLevel

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.0 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.5 kB64.7 kB+178 B (+0.3%)
    • snapshotEqual

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size32.2 kB32.4 kB+177 B (+0.5%)
      size-with-ext-deps80.8 kB81.0 kB+178 B (+0.2%)
    • startAfter

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size34.2 kB34.3 kB+177 B (+0.5%)
      size-with-ext-deps82.7 kB82.9 kB+178 B (+0.2%)
    • startAt

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size34.2 kB34.3 kB+177 B (+0.5%)
      size-with-ext-deps82.7 kB82.9 kB+178 B (+0.2%)
    • terminate

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size16.1 kB16.2 kB+177 B (+1.1%)
      size-with-ext-deps64.7 kB64.9 kB+178 B (+0.3%)
    • updateDoc

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size106 kB106 kB+177 B (+0.2%)
      size-with-ext-deps156 kB156 kB+178 B (+0.1%)
    • waitForPendingWrites

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size86.4 kB86.6 kB+177 B (+0.2%)
      size-with-ext-deps136 kB136 kB+178 B (+0.1%)
    • where

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size40.2 kB40.4 kB+177 B (+0.4%)
      size-with-ext-deps88.8 kB89.0 kB+178 B (+0.2%)
    • writeBatch

      Size

      TypeBase (3fe2c4a)Merge (1aa9044)Diff
      size108 kB108 kB+177 B (+0.2%)
      size-with-ext-deps158 kB158 kB+178 B (+0.1%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/NYo1o42YxF.html

packages/firestore/src/api/credentials.ts Outdated Show resolved Hide resolved
packages/firestore/src/api/credentials.ts Outdated Show resolved Hide resolved
packages/firestore/src/api/credentials.ts Outdated Show resolved Hide resolved
@schmidt-sebastian
Copy link
Contributor

Please add changelog as well

@ehsannas ehsannas merged commit bb8f37c into master Feb 12, 2022
@ehsannas ehsannas deleted the ehsann/appcheck-token-listener branch February 12, 2022 00:28
@google-oss-bot google-oss-bot mentioned this pull request Feb 16, 2022
@firebase firebase locked and limited conversation to collaborators Mar 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants