Skip to content

Commit cb2e5c0

Browse files
committed
refactor!: KeyringController always derives encryption key
1 parent 1149d95 commit cb2e5c0

File tree

5 files changed

+213
-118
lines changed

5 files changed

+213
-118
lines changed

packages/keyring-controller/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1717
- **BREAKING:** The `KeyringController` constructor options now require an encryptor ([#7127](https://github.com/MetaMask/core/pull/7127))
1818
- The `encryptor` constructor option was previously optional and defaulted to an instance of `@metamask/browser-passworder`.
1919
- **BREAKING:** The `GenericEncryptor` and `ExportableKeyEncryptor` types have been merged into a single `Encryptor` type ([#7127](https://github.com/MetaMask/core/pull/7127))
20+
- **BREAKING:** The `Encryptor` type requires `exportKey`, `keyFromPassword` and `generateSalt` methods ([#7127](https://github.com/MetaMask/core/pull/7127))
2021

2122
### Removed
2223

packages/keyring-controller/jest.config.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@ module.exports = merge(baseConfig, {
1717
// An object that configures minimum threshold enforcement for coverage results
1818
coverageThreshold: {
1919
global: {
20-
branches: 94.03,
20+
branches: 93.83,
2121
functions: 100,
22-
lines: 98.31,
23-
statements: 98.33,
22+
lines: 98.12,
23+
statements: 98.14,
2424
},
2525
},
2626

0 commit comments

Comments
 (0)