Skip to content

Commit 9e613d1

Browse files
committed
refactor seedless-onboarding-controller-init encryptor argument
1 parent 65e0b1b commit 9e613d1

File tree

3 files changed

+5
-23
lines changed

3 files changed

+5
-23
lines changed

app/scripts/controller-init/seedless-onboarding/seedless-onboarding-controller-init.test.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,8 @@ describe('SeedlessOnboardingControllerInit', () => {
7171
decryptWithKey: expect.any(Function),
7272
encrypt: expect.any(Function),
7373
encryptWithDetail: expect.any(Function),
74+
encryptWithKey: expect.any(Function),
75+
isVaultUpdated: expect.any(Function),
7476
importKey: expect.any(Function),
7577
exportKey: expect.any(Function),
7678
generateSalt: expect.any(Function),

app/scripts/controller-init/seedless-onboarding/seedless-onboarding-controller-init.ts

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import {
33
SeedlessOnboardingControllerMessenger,
44
Web3AuthNetwork,
55
} from '@metamask/seedless-onboarding-controller';
6-
import { EncryptionKey, EncryptionResult } from '@metamask/browser-passworder';
6+
import { EncryptionKey } from '@metamask/browser-passworder';
77
import { ControllerInitFunction } from '../types';
88
import { encryptorFactory } from '../../lib/encryptor-factory';
99
import { isDevOrTestBuild } from '../../services/oauth/config';
@@ -41,26 +41,7 @@ export const SeedlessOnboardingControllerInit: ControllerInitFunction<
4141
renewRefreshToken: (...args) =>
4242
initMessenger.call('OAuthService:renewRefreshToken', ...args),
4343

44-
encryptor: {
45-
decrypt: encryptor.decrypt,
46-
decryptWithDetail: encryptor.decryptWithDetail,
47-
decryptWithKey: (key: EncryptionKey, encryptedData: EncryptionResult) => {
48-
let payload: EncryptionResult;
49-
if (typeof encryptedData === 'string') {
50-
payload = JSON.parse(encryptedData);
51-
} else {
52-
payload = encryptedData;
53-
}
54-
55-
return encryptor.decryptWithKey(key as EncryptionKey, payload);
56-
},
57-
encrypt: encryptor.encrypt,
58-
encryptWithDetail: encryptor.encryptWithDetail,
59-
importKey: encryptor.importKey,
60-
exportKey: encryptor.exportKey,
61-
keyFromPassword: encryptor.keyFromPassword,
62-
generateSalt: encryptor.generateSalt,
63-
},
44+
encryptor,
6445
});
6546

6647
return {

app/scripts/lib/encryptor-factory.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import {
1313
EncryptionKey,
1414
KeyDerivationOptions,
1515
} from '@metamask/browser-passworder';
16-
import { Encryptor } from '@metamask/keyring-controller';
1716

1817
/**
1918
* A factory function for the encrypt method of the browser-passworder library,
@@ -108,7 +107,7 @@ const isVaultUpdatedFactory = (iterations: number) => (vault: string) =>
108107
* @param iterations - The number of iterations to use for the PBKDF2 algorithm.
109108
* @returns An encryptor set with the given number of iterations.
110109
*/
111-
export const encryptorFactory = (iterations: number): Encryptor => ({
110+
export const encryptorFactory = (iterations: number) => ({
112111
encrypt: encryptFactory(iterations),
113112
encryptWithKey,
114113
encryptWithDetail: encryptWithDetailFactory(iterations),

0 commit comments

Comments
 (0)