Skip to content

Commit de39a5c

Browse files
committed
Merge branch 'master' into fis-compat
2 parents c005c32 + b0f7d4c commit de39a5c

File tree

375 files changed

+13534
-23382
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

375 files changed

+13534
-23382
lines changed

.changeset/large-books-call.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@firebase/auth": patch
3+
---
4+
5+
Update auth token logic to rely on device clock time instead of server time. This fixes an issue seen when a device's clock is skewed by a lot: https://github.com/firebase/firebase-js-sdk/issues/3222

.changeset/rich-birds-wink.md

Lines changed: 0 additions & 4 deletions
This file was deleted.

.changeset/thick-snails-appear.md

Lines changed: 0 additions & 6 deletions
This file was deleted.

.changeset/wicked-actors-clap.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@firebase/firestore": patch
3+
---
4+
5+
Fixes an issue that prevented the SDK from automatically retrieving custom User claims.
Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
---
2+
name: 🧪 Alpha SDK bug report
3+
about: Have you found a bug in one of our pre-release or alpha SDKs? File it here.
4+
title: ""
5+
labels: "alpha"
6+
assignees: ""
7+
---
8+
9+
<!-- Step 1 [READ THIS] -->
10+
<!--
11+
Are you in the right place?
12+
* For issues or feature requests related to __the code in this repository__
13+
file a Github issue.
14+
* If you are not yet participating in our Alpha and would like to join, use this form:
15+
https://forms.gle/bKBEm48W8EMPLaUp9
16+
* For general Alpha discussion, use the google group:
17+
https://groups.google.com/g/firebase-js-sdk-testers
18+
-->
19+
20+
<!-- Step 2 -->
21+
22+
### [REQUIRED] Describe your environment
23+
24+
* Operating System version: _____
25+
* Browser version: _____
26+
* Firebase SDK version: _____
27+
* Firebase Product: _____ (auth, database, storage, etc)
28+
29+
<!-- Step 3 -->
30+
31+
### [REQUIRED] Describe the problem
32+
33+
#### Steps to reproduce:
34+
<!--
35+
What happened? How can we make the problem occur?
36+
This could be a description, log/console output, etc.
37+
-->
38+
#### Relevant Code:
39+
40+
```javascript
41+
// TODO(you): code here to reproduce the problem
42+
```

.github/ISSUE_TEMPLATE.md renamed to .github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,11 @@
1+
---
2+
name: 🐞 Bug report
3+
about: Found a bug in the JS SDK? File it here.
4+
title: ""
5+
labels: ""
6+
assignees: ""
7+
---
8+
19
<!-- Step 1 [READ THIS] -->
210
<!--
311
Are you in the right place?
@@ -33,12 +41,6 @@ Are you in the right place?
3341
-->
3442
#### Relevant Code:
3543

36-
<!--
37-
Reproduce the issue on StackBlitz and provide your forked URL
38-
or give us some sample code below
39-
-->
40-
https://stackblitz.com/fork/firebase-issue-sandbox
41-
4244
```javascript
4345
// TODO(you): code here to reproduce the problem
4446
```

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
blank_issues_enabled: false
2+
contact_links:
3+
- name: 🔥 Firebase Support
4+
url: https://firebase.google.com/support/
5+
about: If you have an urgent issue with your app in production, please contact support.

.github/workflows/test-all.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
- name: yarn build
2727
run: yarn build
2828
- name: Set start timestamp env var
29-
run: echo "::set-env name=FIREBASE_CI_TEST_START_TIME::$(date +%s)"
29+
run: echo "FIREBASE_CI_TEST_START_TIME=$(date +%s)" >> $GITHUB_ENV
3030
- name: Run unit tests
3131
run: |
3232
xvfb-run yarn test:ci

common/api-review/auth-exp.api.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ export function fetchSignInMethodsForEmail(auth: externs.Auth, email: string): P
146146
export function getAdditionalUserInfo(userCredential: externs.UserCredential): externs.AdditionalUserInfo | null;
147147

148148
// @public
149-
export function getAuth(app?: FirebaseApp): Auth;
149+
export function getAuth(app: FirebaseApp): Auth;
150150

151151
// @public
152152
export function getIdToken(user: externs.User, forceRefresh?: boolean): Promise<string>;
@@ -186,7 +186,7 @@ export const indexedDBLocalPersistence: externs.Persistence;
186186
// Warning: (ae-forgotten-export) The symbol "Dependencies" needs to be exported by the entry point index.d.ts
187187
//
188188
// @public (undocumented)
189-
export function initializeAuth(app?: FirebaseApp, deps?: Dependencies): externs.Auth;
189+
export function initializeAuth(app: FirebaseApp, deps?: Dependencies): externs.Auth;
190190

191191
// @public
192192
export const inMemoryPersistence: externs.Persistence;
@@ -247,8 +247,10 @@ export class OAuthProvider implements externs.AuthProvider {
247247
constructor(providerId: string);
248248
addScope(scope: string): externs.AuthProvider;
249249
credential(params: OAuthCredentialOptions): externs.OAuthCredential;
250+
static credentialFromError(error: FirebaseError): externs.OAuthCredential | null;
250251
// (undocumented)
251252
static credentialFromJSON(json: object | string): externs.OAuthCredential;
253+
static credentialFromResult(userCredential: externs.UserCredential): externs.OAuthCredential | null;
252254
// @internal (undocumented)
253255
defaultLanguageCode: string | null;
254256
getCustomParameters(): CustomParameters;
@@ -363,7 +365,7 @@ export function sendPasswordResetEmail(auth: externs.Auth, email: string, action
363365
export function sendSignInLinkToEmail(auth: externs.Auth, email: string, actionCodeSettings?: externs.ActionCodeSettings): Promise<void>;
364366

365367
// @public
366-
export function setPersistence(auth: externs.Auth, persistence: externs.Persistence): void;
368+
export function setPersistence(auth: externs.Auth, persistence: externs.Persistence): Promise<void>;
367369

368370
// @public
369371
export function signInAnonymously(auth: externs.Auth): Promise<externs.UserCredential>;

common/api-review/firestore-exp.api.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import { _FirebaseService } from '@firebase/app-types-exp';
1111
import { LogLevelString as LogLevel } from '@firebase/logger';
1212
import { Provider } from '@firebase/component';
1313
import { SetOptions as SetOptions_2 } from '@firebase/firestore-types';
14-
import { SnapshotMetadata as SnapshotMetadata_2 } from '@firebase/firestore-types';
1514

1615
// @public
1716
export function addDoc<T>(reference: CollectionReference<T>, data: T): Promise<DocumentReference<T>>;
@@ -440,11 +439,11 @@ export interface SnapshotListenOptions {
440439
}
441440

442441
// @public
443-
export class SnapshotMetadata implements SnapshotMetadata_2 {
442+
export class SnapshotMetadata {
444443
constructor(hasPendingWrites: boolean, fromCache: boolean);
445444
readonly fromCache: boolean;
446445
readonly hasPendingWrites: boolean;
447-
isEqual(other: SnapshotMetadata_2): boolean;
446+
isEqual(other: SnapshotMetadata): boolean;
448447
}
449448

450449
// @public

common/api-review/firestore-lite.api.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,8 +168,10 @@ export abstract class FieldValue {
168168
abstract _toFieldTransform(context: ParseContext): FieldTransform | null;
169169
}
170170

171+
// Warning: (ae-forgotten-export) The symbol "FirestoreService" needs to be exported by the entry point index.d.ts
172+
//
171173
// @public
172-
export class FirebaseFirestore implements _FirebaseService {
174+
export class FirebaseFirestore implements FirestoreService {
173175
constructor(databaseIdOrApp: DatabaseId | FirebaseApp, authProvider: Provider<FirebaseAuthInternalName>);
174176
get app(): FirebaseApp;
175177
// Warning: (ae-forgotten-export) The symbol "CredentialsProvider" needs to be exported by the entry point index.d.ts

0 commit comments

Comments
 (0)