@metamask-previews/seedless-onboarding-controller 7.1.0-preview-77d255e6a → 7.1.0-preview-fb7aa07ff

package/CHANGELOG.md

@@ -7,14 +7,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-### Added
-
-- **BREAKING** The `encryptor` constructor param requires `encryptWithKey` method. ([#7800](https://github.com/MetaMask/core/pull/7800))
-  - The method is to encrypt the vault with cached encryption key while the wallet is unlocked.
-- Added new public method, `getAccessToken`. ([#7800](https://github.com/MetaMask/core/pull/7800))
-  - Clients can use this method to get `accessToken` from the controller, instead of directly accessing from the state.
-  - This method also adds refresh token mechanism when `accessToken` is expired, hence preventing expired token usage in the clients.
-
 ### Changed
 
 - Update StateMetadata's `includeInStateLogs` property. ([#7750](https://github.com/MetaMask/core/pull/7750))
@@ -22,10 +14,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Bump `@metamask/keyring-controller` from `^25.0.0` to `^25.1.0` ([#7713](https://github.com/MetaMask/core/pull/7713))
 - Upgrade `@metamask/utils` from `^11.8.1` to `^11.9.0` ([#7511](https://github.com/MetaMask/core/pull/7511))
 
-### Fixed
-
-- Fixed new `accessToken` not being persisted in the vault after the token refresh. ([#7800](https://github.com/MetaMask/core/pull/7800))
-
 ## [7.1.0]
 
 ### Added

package/dist/SeedlessOnboardingController.cjs

@@ -10,12 +10,11 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
     if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
     return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
 };
-var _SeedlessOnboardingController_instances, _SeedlessOnboardingController_vaultEncryptor, _SeedlessOnboardingController_controllerOperationMutex, _SeedlessOnboardingController_vaultOperationMutex, _SeedlessOnboardingController_refreshJWTToken, _SeedlessOnboardingController_revokeRefreshToken, _SeedlessOnboardingController_renewRefreshToken, _SeedlessOnboardingController_passwordOutdatedCacheTTL, _SeedlessOnboardingController_isUnlocked, _SeedlessOnboardingController_cachedDecryptedVaultData, _SeedlessOnboardingController_submitGlobalPassword, _SeedlessOnboardingController_setUnlocked, _SeedlessOnboardingController_pickLatestAccessToken, _SeedlessOnboardingController_persistOprfKey, _SeedlessOnboardingController_persistAuthPubKey, _SeedlessOnboardingController_storeKeyringEncryptionKey, _SeedlessOnboardingController_loadKeyringEncryptionKey, _SeedlessOnboardingController_loadSeedlessEncryptionKey, _SeedlessOnboardingController_recoverAuthPubKey, _SeedlessOnboardingController_recoverEncKey, _SeedlessOnboardingController_fetchAllSecretDataFromMetadataStore, _SeedlessOnboardingController_changeEncryptionKey, _SeedlessOnboardingController_encryptAndStoreSecretData, _SeedlessOnboardingController_unlockVaultAndGetVaultData, _SeedlessOnboardingController_decryptAndParseVaultData, _SeedlessOnboardingController_withPersistedSecretMetadataBackupsState, _SeedlessOnboardingController_filterDupesAndUpdateSocialBackupsMetadata, _SeedlessOnboardingController_createNewVaultWithAuthData, _SeedlessOnboardingController_updateVault, _SeedlessOnboardingController_getAccessTokenAndRevokeToken, _SeedlessOnboardingController_withControllerLock, _SeedlessOnboardingController_withVaultLock, _SeedlessOnboardingController_parseVaultData, _SeedlessOnboardingController_assertIsUnlocked, _SeedlessOnboardingController_assertIsAuthenticatedUser, _SeedlessOnboardingController_assertPasswordInSync, _SeedlessOnboardingController_resetPasswordOutdatedCache, _SeedlessOnboardingController_addRefreshTokenToRevokeList, _SeedlessOnboardingController_isAuthTokenError, _SeedlessOnboardingController_isMaxKeyChainLengthError, _SeedlessOnboardingController_executeWithTokenRefresh, _SeedlessOnboardingController_updateVaultAfterAuthTokenRefresh, _SeedlessOnboardingController_checkTokensExpired;
+var _SeedlessOnboardingController_instances, _SeedlessOnboardingController_vaultEncryptor, _SeedlessOnboardingController_controllerOperationMutex, _SeedlessOnboardingController_vaultOperationMutex, _SeedlessOnboardingController_refreshJWTToken, _SeedlessOnboardingController_revokeRefreshToken, _SeedlessOnboardingController_renewRefreshToken, _SeedlessOnboardingController_passwordOutdatedCacheTTL, _SeedlessOnboardingController_isUnlocked, _SeedlessOnboardingController_cachedDecryptedVaultData, _SeedlessOnboardingController_submitGlobalPassword, _SeedlessOnboardingController_setUnlocked, _SeedlessOnboardingController_persistOprfKey, _SeedlessOnboardingController_persistAuthPubKey, _SeedlessOnboardingController_storeKeyringEncryptionKey, _SeedlessOnboardingController_loadKeyringEncryptionKey, _SeedlessOnboardingController_loadSeedlessEncryptionKey, _SeedlessOnboardingController_recoverAuthPubKey, _SeedlessOnboardingController_recoverEncKey, _SeedlessOnboardingController_fetchAllSecretDataFromMetadataStore, _SeedlessOnboardingController_changeEncryptionKey, _SeedlessOnboardingController_encryptAndStoreSecretData, _SeedlessOnboardingController_unlockVaultAndGetVaultData, _SeedlessOnboardingController_decryptAndParseVaultData, _SeedlessOnboardingController_withPersistedSecretMetadataBackupsState, _SeedlessOnboardingController_filterDupesAndUpdateSocialBackupsMetadata, _SeedlessOnboardingController_createNewVaultWithAuthData, _SeedlessOnboardingController_updateVault, _SeedlessOnboardingController_getAccessTokenAndRevokeToken, _SeedlessOnboardingController_withControllerLock, _SeedlessOnboardingController_withVaultLock, _SeedlessOnboardingController_parseVaultData, _SeedlessOnboardingController_assertIsUnlocked, _SeedlessOnboardingController_assertIsAuthenticatedUser, _SeedlessOnboardingController_assertPasswordInSync, _SeedlessOnboardingController_resetPasswordOutdatedCache, _SeedlessOnboardingController_addRefreshTokenToRevokeList, _SeedlessOnboardingController_isAuthTokenError, _SeedlessOnboardingController_isMaxKeyChainLengthError, _SeedlessOnboardingController_executeWithTokenRefresh;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SeedlessOnboardingController = exports.getInitialSeedlessOnboardingControllerStateWithDefaults = void 0;
 const auth_network_utils_1 = require("@metamask/auth-network-utils");
 const base_controller_1 = require("@metamask/base-controller");
-const messenger_1 = require("@metamask/messenger");
 const toprf_secure_backup_1 = require("@metamask/toprf-secure-backup");
 const utils_1 = require("@metamask/utils");
 const aes_1 = require("@noble/ciphers/aes");
@@ -243,7 +242,6 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
         __classPrivateFieldSet(this, _SeedlessOnboardingController_refreshJWTToken, refreshJWTToken, "f");
         __classPrivateFieldSet(this, _SeedlessOnboardingController_revokeRefreshToken, revokeRefreshToken, "f");
         __classPrivateFieldSet(this, _SeedlessOnboardingController_renewRefreshToken, renewRefreshToken, "f");
-        this.messenger.registerActionHandler('SeedlessOnboardingController:getAccessToken', this.getAccessToken.bind(this));
     }
     async fetchMetadataAccessCreds() {
         const { metadataAccessToken } = this.state;
@@ -554,7 +552,7 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
     /**
      * Submit the password to the controller, verify the password validity and unlock the controller.
      *
-     * This method will be used especially when user unlock the wallet.
+     * This method will be used especially when user rehydrate/unlock the wallet.
      * The provided password will be verified against the encrypted vault, encryption key will be derived and saved in the controller state.
      *
      * This operation is useful when user performs some actions that requires the user password/encryption key. e.g. add new srp backup
@@ -564,27 +562,7 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
      */
     async submitPassword(password) {
         return await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_withControllerLock).call(this, async () => {
-            // get the access token from the state before unlocking, it might be the new token set from the `refreshAuthTokens` method.
-            const { accessToken: accessTokenBeforeUnlock } = this.state;
-            const deserializedVaultData = await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_unlockVaultAndGetVaultData).call(this, {
-                password,
-            });
-            const accessTokenFromDecryptedVault = deserializedVaultData.accessToken;
-            // Pick the latest access token - the token from state might be newer (from refreshAuthTokens)
-            // than the token stored in the vault.
-            const latestAccessToken = __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_pickLatestAccessToken).call(this, accessTokenBeforeUnlock, accessTokenFromDecryptedVault);
-            // update the state and vault with the latest access token `ONLY` if it's different from the current access token in the state.
-            if (latestAccessToken !== accessTokenFromDecryptedVault) {
-                const updatedVaultData = {
-                    ...deserializedVaultData,
-                    accessToken: latestAccessToken,
-                };
-                await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_updateVault).call(this, {
-                    password,
-                    vaultData: updatedVaultData,
-                    pwEncKey: deserializedVaultData.toprfPwEncryptionKey,
-                });
-            }
+            await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_unlockVaultAndGetVaultData).call(this, { password });
             __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_setUnlocked).call(this);
         });
     }
@@ -790,8 +768,6 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
                 refreshToken,
                 skipLock: true,
             });
-            // update the vault with new access token
-            await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_updateVaultAfterAuthTokenRefresh).call(this, accessToken);
         }
         catch (error) {
             log('Error refreshing node auth tokens', error);
@@ -882,21 +858,6 @@ class SeedlessOnboardingController extends base_controller_1.BaseController {
             }
         });
     }
-    /**
-     * Get the access token from the state.
-     *
-     * If the tokens are expired, the method will refresh them and return the new access token.
-     *
-     * @returns The access token.
-     */
-    async getAccessToken() {
-        return __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_withControllerLock).call(this, async () => {
-            __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_assertIsAuthenticatedUser).call(this, this.state);
-            return __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_executeWithTokenRefresh).call(this, async () => {
-                return this.state.accessToken;
-            }, 'getAccessToken');
-        });
-    }
     /**
      * Check if the current node auth token is expired.
      *
@@ -965,28 +926,22 @@ _SeedlessOnboardingController_vaultEncryptor = new WeakMap(), _SeedlessOnboardin
  * corresponding to the current authPubKey in state.
  */
 async function _SeedlessOnboardingController_submitGlobalPassword({ targetAuthPubKey, globalPassword, maxKeyChainLength, }) {
-    const { pwEncKey: globalPwEncKey, authKeyPair: globalAuthKeyPair } = await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_recoverEncKey).call(this, globalPassword);
+    const { pwEncKey: curPwEncKey, authKeyPair: curAuthKeyPair } = await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_recoverEncKey).call(this, globalPassword);
     try {
         // Recover vault encryption key.
         const res = await this.toprfClient.recoverPwEncKey({
             targetAuthPubKey,
-            curPwEncKey: globalPwEncKey,
-            curAuthKeyPair: globalAuthKeyPair,
+            curPwEncKey,
+            curAuthKeyPair,
             maxPwChainLength: maxKeyChainLength,
         });
         const { pwEncKey } = res;
         const vaultKey = await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_loadSeedlessEncryptionKey).call(this, pwEncKey);
-        // accessToken before unlocking vault and flooding the state with values from the decrypted vault
-        // it might be the new token set from the `refreshAuthTokens` method.
-        const { accessToken: accessTokenBeforeUnlock } = this.state;
         // Unlock the controller
-        const decryptedVaultData = await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_unlockVaultAndGetVaultData).call(this, {
+        await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_unlockVaultAndGetVaultData).call(this, {
             encryptionKey: vaultKey,
         });
         __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_setUnlocked).call(this);
-        // Pick the latest access token - the token from state might be newer (from refreshAuthTokens)
-        // than the token stored in the vault. The vault will be updated later by syncLatestGlobalPassword.
-        __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_pickLatestAccessToken).call(this, accessTokenBeforeUnlock, decryptedVaultData.accessToken);
     }
     catch (error) {
         if (__classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_isAuthTokenError).call(this, error)) {
@@ -999,18 +954,6 @@ async function _SeedlessOnboardingController_submitGlobalPassword({ targetAuthPu
     }
 }, _SeedlessOnboardingController_setUnlocked = function _SeedlessOnboardingController_setUnlocked() {
     __classPrivateFieldSet(this, _SeedlessOnboardingController_isUnlocked, true, "f");
-}, _SeedlessOnboardingController_pickLatestAccessToken = function _SeedlessOnboardingController_pickLatestAccessToken(tokenBeforeUnlock, tokenAfterUnlock) {
-    let latestToken = tokenAfterUnlock;
-    if (tokenBeforeUnlock &&
-        tokenAfterUnlock &&
-        tokenBeforeUnlock !== tokenAfterUnlock) {
-        latestToken = (0, utils_3.compareAndGetLatestToken)(tokenBeforeUnlock, tokenAfterUnlock);
-        // Update the access token in the state with the latest access token
-        this.update((state) => {
-            state.accessToken = latestToken;
-        });
-    }
-    return latestToken;
 }, _SeedlessOnboardingController_persistOprfKey = 
 /**
  * Persist the encryption key for the seedless onboarding flow.
@@ -1428,14 +1371,14 @@ async function _SeedlessOnboardingController_updateVault({ password, vaultData,
         // from the password using an intentionally slow key derivation function.
         // We should make sure that we only call it very intentionally.
         const { vault, exportedKeyString } = await __classPrivateFieldGet(this, _SeedlessOnboardingController_vaultEncryptor, "f").encryptWithDetail(password, serializedVaultData);
+        // Encrypt vault key.
         const aes = (0, webcrypto_1.managedNonce)(aes_1.gcm)(pwEncKey);
         const encryptedKey = aes.encrypt((0, utils_2.utf8ToBytes)(exportedKeyString));
-        const encryptedSeedlessEncryptionKey = (0, utils_1.bytesToBase64)(encryptedKey);
         this.update((state) => {
             state.vault = vault;
             state.vaultEncryptionKey = exportedKeyString;
             state.vaultEncryptionSalt = JSON.parse(vault).salt;
-            state.encryptedSeedlessEncryptionKey = encryptedSeedlessEncryptionKey;
+            state.encryptedSeedlessEncryptionKey = (0, utils_1.bytesToBase64)(encryptedKey);
         });
     });
 }, _SeedlessOnboardingController_getAccessTokenAndRevokeToken = 
@@ -1594,7 +1537,18 @@ async function _SeedlessOnboardingController_assertPasswordInSync(options) {
  */
 async function _SeedlessOnboardingController_executeWithTokenRefresh(operation, operationName) {
     try {
-        if (__classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_checkTokensExpired).call(this)) {
+        // proactively check for expired tokens and refresh them if needed
+        const isNodeAuthTokenExpired = this.checkNodeAuthTokenExpired();
+        const isMetadataAccessTokenExpired = this.checkMetadataAccessTokenExpired();
+        // access token is only accessible when the vault is unlocked
+        // so skip the check if the vault is locked
+        let isAccessTokenExpired = false;
+        if (__classPrivateFieldGet(this, _SeedlessOnboardingController_isUnlocked, "f")) {
+            isAccessTokenExpired = this.checkAccessTokenExpired();
+        }
+        if (isNodeAuthTokenExpired ||
+            isMetadataAccessTokenExpired ||
+            isAccessTokenExpired) {
             log(`JWT token expired during ${operationName}, attempting to refresh tokens`, 'node auth token exp check');
             await this.refreshAuthTokens();
         }
@@ -1620,58 +1574,6 @@ async function _SeedlessOnboardingController_executeWithTokenRefresh(operation,
             throw error;
         }
     }
-}, _SeedlessOnboardingController_updateVaultAfterAuthTokenRefresh = async function _SeedlessOnboardingController_updateVaultAfterAuthTokenRefresh(accessToken) {
-    await __classPrivateFieldGet(this, _SeedlessOnboardingController_instances, "m", _SeedlessOnboardingController_withVaultLock).call(this, async () => {
-        if (!__classPrivateFieldGet(this, _SeedlessOnboardingController_isUnlocked, "f")) {
-            // we just temporarily store the access token in the state
-            // when user attempts to unlock the vault, we will use this access token to update the vault
-            this.update((state) => {
-                state.accessToken = accessToken;
-            });
-            return;
-        }
-        const { vaultEncryptionKey, vaultEncryptionSalt } = this.state;
-        if (!vaultEncryptionKey ||
-            !vaultEncryptionSalt ||
-            !__classPrivateFieldGet(this, _SeedlessOnboardingController_cachedDecryptedVaultData, "f")) {
-            throw new Error(constants_1.SeedlessOnboardingControllerErrorMessage.MissingCredentials);
-        }
-        const serializedVaultData = (0, utils_3.serializeVaultData)({
-            ...__classPrivateFieldGet(this, _SeedlessOnboardingController_cachedDecryptedVaultData, "f"),
-            accessToken,
-        });
-        const encryptionKey = await __classPrivateFieldGet(this, _SeedlessOnboardingController_vaultEncryptor, "f").importKey(vaultEncryptionKey);
-        const updatedEncVault = await __classPrivateFieldGet(this, _SeedlessOnboardingController_vaultEncryptor, "f").encryptWithKey(encryptionKey, serializedVaultData);
-        // NOTE: Referenced from keyring-controller!
-        // We need to include the salt used to derive
-        // the encryption key, to be able to derive it
-        // from password again.
-        updatedEncVault.salt = vaultEncryptionSalt;
-        this.update((state) => {
-            state.vault = JSON.stringify(updatedEncVault);
-            state.vaultEncryptionSalt = vaultEncryptionSalt;
-            state.accessToken = accessToken;
-            state.vaultEncryptionKey = vaultEncryptionKey;
-        });
-        // update the cached decrypted vault data with the new access token
-        __classPrivateFieldSet(this, _SeedlessOnboardingController_cachedDecryptedVaultData, {
-            ...__classPrivateFieldGet(this, _SeedlessOnboardingController_cachedDecryptedVaultData, "f"),
-            accessToken,
-        }, "f");
-    });
-}, _SeedlessOnboardingController_checkTokensExpired = function _SeedlessOnboardingController_checkTokensExpired() {
-    // proactively check for expired tokens and refresh them if needed
-    const isNodeAuthTokenExpired = this.checkNodeAuthTokenExpired();
-    const isMetadataAccessTokenExpired = this.checkMetadataAccessTokenExpired();
-    // access token is only accessible when the vault is unlocked
-    // so skip the check if the vault is locked
-    let isAccessTokenExpired = false;
-    if (__classPrivateFieldGet(this, _SeedlessOnboardingController_isUnlocked, "f")) {
-        isAccessTokenExpired = this.checkAccessTokenExpired();
-    }
-    return (isNodeAuthTokenExpired ||
-        isMetadataAccessTokenExpired ||
-        isAccessTokenExpired);
 };
 /**
  * Assert that the provided password is a valid non-empty string.