Skip to content

Tomandersen/session token persistence #8390

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

Closed
wants to merge 285 commits into from

Conversation

tom-andersen
Copy link
Contributor

Persistence of SessionToken

hsubox76 and others added 30 commits July 24, 2024 10:55
* added authStateReady function

* updated Auth interface

* added unit tests on authStateReady

* updated demo app

* added authStateReady to Auth interface

* fixed formatting issue

* updated authStateReady calls in demo app

* fixed formatting issues

* fixed demo app to incorporate authStateReady

* formatted code

* removed unnecessary async keywords

* reverted changes in onSignOut

* clean up code

* fixed comments

* resolved code review comments and updated tests

* changed reference doc

* resolved comments from pr

* added changeset

* Update twelve-actors-enjoy.md

* resolved doc change check failure

* resolved issues with Doc Change Check

* clarify comments

* fixed doc change check issue
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Expose PrivateSettings interface as an internal type for the Firestore console. It was exported in the previous version of the non-modular firestore SDK we used in console.
…message in deep equals (#7519)

* equality_matcher.ts: fix absence of custom message in assertion failure messages

* equality_matcher.ts: remove usage of the now-deleted `msg` argument

* equality_matcher.ts: fix type of `originalFunction`
Fix aggregate query order-by normalization to support future aggregate operations.
…uth (#7514)

* Implement validatePassword endpoint for public API with PasswordPolicy and PasswordValidationStatus public types

* Update auth demo to include a section for password validation
* Update proto loader

* Update proto loader

* Update proto loader

* Update proto loader
Review and update of the CONTRIBUTING.md guide.
* Fix and skip flaky tests

* add todos

* more nits
* Fix auth event uncancellable bug

* add changeset file

* includes issue number on changesets

* yarn format
hsubox76 and others added 23 commits July 24, 2024 10:55
* fix: Fix a typo in the webchannel-wrapper's package.json.

* Create orange-rings-own.md
* Update sync_engine_impl.ts

* add spec test

* update comment

* typo

* update spec test

* add changeset
Update browser detection logic: Detect either window or web worker (WorkerGlobalScope).
This updates the implementation to `isBrowser()` and adds a new function `isWebWorker()`.

Fixes #8299 #8284
… availability. (#8335)

There was code that would errantly attempt to allocate a FinalizationRegstiry in FirebaseServerAppImpl regardless of the existence of releaseOnDeref field, and this caused issues in some edge environments.

This change fixes that issue.
The custom domain "firebase-sdk-health-metrics.com" currently
associated with the Metric Service will be deprecated eventually,
due to the acquisition of Google Domains by Squarespace.

Switching to the original URL created by Google Cloud Run (which
hosts the Metric Service) to avoid service disruption.

See b/303931111 for more details.
)

This change fixes a TypeScript compilation error. 

The `FirebaseServerAppSettings.name` field inherited from `FirebaseAppSettings` but the type was redefined from `string` to `undefined`. This redefinition would cause a TypeScript compliation error if `exactOptionalPropertyTypes` was set `true` in `packages/app/tsconfig.json`.

This change now uses `omit< , >` to strip the `name` field from the `FirebaseServerAppSettings` declaration, where `FirebaseAppSettings` is extended.

Fixes #8336
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ng yarn lock (#8298)

* Upgrade google-cloud-firestore in rules-unit-testing yarn lock

The version of google-cloud-firestore in the rules-unit-testing
yarn.lock was listed as 6.6.0, when it was actually 6.8.0. This change
ugrades the listed version to be the real version.

* Add changeset
* Use safevalues to fix trusted types issues reported by tsec

* Upgrade to safevalues 0.6.0

* Remove exemptions, and untested usages of safevalues

* Add dependency that was accidentally removed

* Add FIXMEs for tsec violations

* Run formatting

* Compare against full Gtag script in tests

* Check that full reCAPTCHA script URL was assigned to script element

* Replace FIXMEs with TODOs

* Remove auth, rtdb, messaging from changeset
Numerous fixes to spelling across the SDK.
Fixed the format of a changeset file. I had hand edited a changeset in a previous PR and accidentally stripped out the "release notes" section.
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@tom-andersen tom-andersen requested review from a team as code owners July 24, 2024 15:44
Copy link

changeset-bot bot commented Jul 24, 2024

⚠️ No Changeset found

Latest commit: da9d2de

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • @firebase/firestore

    TypeBase (46da093)Merge (7af0691)Diff
    browser378 kB379 kB+896 B (+0.2%)
    esm5363 kB364 kB+932 B (+0.3%)
    main582 kB583 kB+1.25 kB (+0.2%)
    module378 kB379 kB+896 B (+0.2%)
    react-native378 kB379 kB+896 B (+0.2%)
  • bundle

    TypeBase (46da093)Merge (7af0691)Diff
    firestore (CSI Auto Indexing Disable and Delete)268 kB269 kB+687 B (+0.3%)
    firestore (CSI Auto Indexing Enable)268 kB269 kB+687 B (+0.3%)
    firestore (Persistence)303 kB304 kB+687 B (+0.2%)
    firestore (Query Cursors)240 kB240 kB+228 B (+0.1%)
    firestore (Query)238 kB238 kB+228 B (+0.1%)
    firestore (Read data once)226 kB226 kB+228 B (+0.1%)
    firestore (Read Write w Persistence)323 kB323 kB+687 B (+0.2%)
    firestore (Realtime updates)228 kB228 kB+228 B (+0.1%)
    firestore (Transaction)205 kB205 kB+228 B (+0.1%)
    firestore (Write data)205 kB205 kB+228 B (+0.1%)
  • firebase

    TypeBase (46da093)Merge (7af0691)Diff
    firebase-compat.js789 kB790 kB+662 B (+0.1%)
    firebase-firestore-compat.js341 kB342 kB+662 B (+0.2%)
    firebase-firestore.js437 kB438 kB+896 B (+0.2%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

Size Analysis Report 1

This report is too large (308,771 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

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

@firebase firebase locked and limited conversation to collaborators Aug 24, 2024
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.