Merge remote-tracking branch 'origin/auth-next' into fei-auth-next-build

Author: Feiyang1

packages-exp/auth-compat-exp/index.ts

@@ -16,19 +16,19 @@
  */
 
 import firebase from '@firebase/app';
-import * as impl from '@firebase/auth-exp/internal';
 import { _FirebaseNamespace } from '@firebase/app-types/private';
+import * as impl from '@firebase/auth-exp/internal';
 import * as externs from '@firebase/auth-types-exp';
 import {
   Component,
   ComponentType,
   InstantiationMode
 } from '@firebase/component';
-import '@firebase/installations';
+
 import { version } from './package.json';
 import { Auth } from './src/auth';
-import { RecaptchaVerifier } from './src/recaptcha_verifier';
 import { Persistence } from './src/persistence';
+import { RecaptchaVerifier } from './src/recaptcha_verifier';
 
 const AUTH_TYPE = 'auth';
 

packages-exp/auth-compat-exp/src/auth.ts

@@ -15,24 +15,26 @@
  * limitations under the License.
  */
 
+import '@firebase/installations';
+
 import { FirebaseApp } from '@firebase/app-types';
 import * as impl from '@firebase/auth-exp/internal';
 import * as compat from '@firebase/auth-types';
 import * as externs from '@firebase/auth-types-exp';
-import '@firebase/installations';
 import {
-  Observer,
-  Unsubscribe,
   ErrorFn,
-  isIndexedDBAvailable
+  isIndexedDBAvailable,
+  Observer,
+  Unsubscribe
 } from '@firebase/util';
+
+import { _validatePersistenceArgument, Persistence } from './persistence';
+import { _getClientPlatform, _isPopupRedirectSupported } from './platform';
 import { User } from './user';
 import {
   convertConfirmationResult,
   convertCredential
 } from './user_credential';
-import { _isPopupRedirectSupported, _getClientPlatform } from './platform';
-import { Persistence, _validatePersistenceArgument } from './persistence';
 
 export class Auth extends impl.AuthImplCompat<User>
   implements compat.FirebaseAuth {
@@ -59,7 +61,7 @@ export class Auth extends impl.AuthImplCompat<User>
       sdkClientVersion: impl._getClientVersion(_getClientPlatform())
     };
 
-    super(app.name, config, User);
+    super(app, config, User);
     this.app = app;
 
     // This promise is intended to float; auth initialization happens in the

packages-exp/auth-exp/demo/src/index.js

@@ -33,6 +33,7 @@ import {
   fetchSignInMethodsForEmail,
   indexedDBLocalPersistence,
   initializeAuth,
+  getAuth,
   inMemoryPersistence,
   isSignInWithEmailLink,
   linkWithCredential,
@@ -1640,10 +1641,7 @@ function onApplyAuthSettingsChange() {
 function initApp() {
   log('Initializing app...');
   app = initializeApp(config);
-  auth = initializeAuth(app, {
-    persistence: browserSessionPersistence,
-    popupRedirectResolver: browserPopupRedirectResolver
-  });
+  auth = getAuth(app);
 
   tempApp = initializeApp(
     {

packages-exp/auth-exp/demo/src/worker/service-worker.ts

@@ -21,8 +21,9 @@
  * mode.
  */
 import { initializeApp } from '@firebase/app-exp';
-import { initializeAuth, indexedDBLocalPersistence } from '@firebase/auth-exp';
+import { getAuth } from '@firebase/auth-exp';
 import { User } from '@firebase/auth-types-exp';
+
 import { config } from '../config';
 
 declare let self: ServiceWorkerGlobalScope;
@@ -38,9 +39,7 @@ const urlsToCache = [
 
 // Initialize the Firebase app in the service worker.
 const app = initializeApp(config);
-const auth = initializeAuth(app, {
-  persistence: indexedDBLocalPersistence
-});
+const auth = getAuth(app);
 
 /**
  * Returns a promise that resolves with an ID token if available.

packages-exp/auth-exp/demo/src/worker/web-worker.ts

@@ -18,9 +18,8 @@ import { initializeApp } from '@firebase/app-exp';
 import {
   createUserWithEmailAndPassword,
   fetchSignInMethodsForEmail,
+  getAuth,
   GoogleAuthProvider,
-  initializeAuth,
-  inMemoryPersistence,
   signInAnonymously,
   signInWithCredential,
   signInWithEmailAndPassword,
@@ -37,9 +36,7 @@ import { config } from '../config';
 
 // Initialize the Firebase app in the web worker.
 const app = initializeApp(config);
-const auth = initializeAuth(app, {
-  persistence: inMemoryPersistence
-});
+const auth = getAuth(app);
 
 /**
  * Returns a promise that resolves with an ID token if available.

packages-exp/auth-exp/index.node.ts

@@ -24,11 +24,13 @@
 
 import * as fetchImpl from 'node-fetch';
 
-import { registerVersion } from '@firebase/app-exp';
-import { name, version } from './package.json';
-import { _initializeAuthForClientPlatform } from './src/core/auth/auth_impl';
-import { ClientPlatform } from './src/core/util/version';
+import { FirebaseApp } from '@firebase/app-types-exp';
+import { Auth } from '@firebase/auth-types-exp';
+
+import { initializeAuth } from './src';
+import { registerAuth } from './src/core/auth/register';
 import { FetchProvider } from './src/core/util/fetch_provider';
+import { ClientPlatform } from './src/core/util/version';
 
 // Initialize the fetch polyfill, the types are slightly off so just cast and hope for the best
 FetchProvider.initialize(
@@ -40,8 +42,8 @@ FetchProvider.initialize(
 // Core functionality shared by all clients
 export * from './src';
 
-export const initializeAuth = _initializeAuthForClientPlatform(
-  ClientPlatform.NODE
-);
+export function getAuth(app?: FirebaseApp): Auth {
+  return initializeAuth(app);
+}
 
-registerVersion(name, version, 'node');
+registerAuth(ClientPlatform.NODE);

packages-exp/auth-exp/index.rn.ts

@@ -22,12 +22,15 @@
  * just use index.ts
  */
 
-import { registerVersion } from '@firebase/app-exp';
 import { AsyncStorage } from 'react-native';
-import { name, version } from './package.json';
-import { _initializeAuthForClientPlatform } from './src/core/auth/auth_impl';
-import { getReactNativePersistence } from './src/platform_react_native/persistence/react_native';
+
+import { FirebaseApp } from '@firebase/app-types-exp';
+import { Auth } from '@firebase/auth-types-exp';
+
+import { initializeAuth } from './src';
+import { registerAuth } from './src/core/auth/register';
 import { ClientPlatform } from './src/core/util/version';
+import { getReactNativePersistence } from './src/platform_react_native/persistence/react_native';
 
 // Core functionality shared by all clients
 export * from './src';
@@ -36,8 +39,10 @@ export const reactNativeLocalPersistence = getReactNativePersistence(
   AsyncStorage
 );
 
-export const initializeAuth = _initializeAuthForClientPlatform(
-  ClientPlatform.REACT_NATIVE
-);
+export function getAuth(app?: FirebaseApp): Auth {
+  return initializeAuth(app, {
+    persistence: reactNativeLocalPersistence
+  });
+}
 
-registerVersion(name, version, 'rn');
+registerAuth(ClientPlatform.REACT_NATIVE);

packages-exp/auth-exp/index.ts

@@ -15,21 +15,24 @@
  * limitations under the License.
  */
 
-import { registerVersion } from '@firebase/app-exp';
-import { name, version } from './package.json';
-import { _initializeAuthForClientPlatform } from './src/core/auth/auth_impl';
+import { FirebaseApp } from '@firebase/app-types-exp';
+import { Auth } from '@firebase/auth-types-exp';
+
+import { initializeAuth } from './src';
+import { registerAuth } from './src/core/auth/register';
 import { ClientPlatform } from './src/core/util/version';
+import { browserLocalPersistence } from './src/platform_browser/persistence/local_storage';
+import { indexedDBLocalPersistence } from './src/platform_browser/persistence/indexed_db';
+import { browserPopupRedirectResolver } from './src/platform_browser/popup_redirect';
 
 // Core functionality shared by all clients
 export * from './src';
 
 // Additional DOM dependend functionality
 
 // persistence
-export {
-  browserLocalPersistence,
-  browserSessionPersistence
-} from './src/platform_browser/persistence/browser';
+export { browserLocalPersistence } from './src/platform_browser/persistence/local_storage';
+export { browserSessionPersistence } from './src/platform_browser/persistence/session_storage';
 export { indexedDBLocalPersistence } from './src/platform_browser/persistence/indexed_db';
 
 // providers
@@ -60,8 +63,11 @@ export { browserPopupRedirectResolver } from './src/platform_browser/popup_redir
 // MFA
 export { PhoneMultiFactorGenerator } from './src/platform_browser/mfa/assertions/phone';
 
-export const initializeAuth = _initializeAuthForClientPlatform(
-  ClientPlatform.BROWSER
-);
+export function getAuth(app?: FirebaseApp): Auth {
+  return initializeAuth(app, {
+    popupRedirectResolver: browserPopupRedirectResolver,
+    persistence: [indexedDBLocalPersistence, browserLocalPersistence]
+  });
+}
 
-registerVersion(name, version);
+registerAuth(ClientPlatform.BROWSER);

packages-exp/auth-exp/index.webworker.ts

@@ -15,16 +15,42 @@
  * limitations under the License.
  */
 
-import { registerVersion } from '@firebase/app-exp';
-import { name, version } from './package.json';
-import { _initializeAuthForClientPlatform } from './src/core/auth/auth_impl';
+import { _getProvider, getApp } from '@firebase/app-exp';
+import { Auth } from '@firebase/auth-types-exp';
+
+import { AuthImpl } from './src/core/auth/auth_impl';
+import { _initializeAuthInstance } from './src/core/auth/initialize';
+import { _AUTH_COMPONENT_NAME, registerAuth } from './src/core/auth/register';
+import { Persistence } from './src/core/persistence';
+import { _getInstance } from './src/core/util/instantiator';
 import { ClientPlatform } from './src/core/util/version';
+import { indexedDBLocalPersistence } from './src/platform_browser/persistence/indexed_db';
 
 // Core functionality shared by all clients
 export * from './src';
 
-export const initializeAuth = _initializeAuthForClientPlatform(
-  ClientPlatform.WORKER
-);
+// persistence
+export { indexedDBLocalPersistence } from './src/platform_browser/persistence/indexed_db';
+
+registerAuth(ClientPlatform.WORKER);
+
+export function getAuth(app = getApp()): Auth {
+  // Unlike the other environments, we need to explicitly check if indexedDb is
+  // available. That means doing the whole rigamarole
+  const auth = _getProvider(
+    app,
+    _AUTH_COMPONENT_NAME
+  ).getImmediate() as AuthImpl;
+
+  // This promise is intended to float; auth initialization happens in the
+  // background, meanwhile the auth object may be used by the app.
+  // eslint-disable-next-line @typescript-eslint/no-floating-promises
+  _getInstance<Persistence>(indexedDBLocalPersistence)
+    ._isAvailable()
+    .then(avail => {
+      const deps = avail ? { persistence: indexedDBLocalPersistence } : {};
+      _initializeAuthInstance(auth, deps);
+    });
 
-registerVersion(name, version, 'webworker');
+  return auth;
+}

packages-exp/auth-exp/package.json

@@ -39,6 +39,7 @@
   "dependencies": {
     "@firebase/logger": "0.2.6",
     "@firebase/util": "0.3.0",
+    "@firebase/component": "0.1.17",
     "@firebase/auth-types-exp": "0.0.800",
     "node-fetch": "2.6.0",
     "tslib": "^1.11.1"