Skip to content

Commit 0cef4aa

Browse files
mikespositodbrans
authored andcommitted
fix: bump @metamask/keyring-controller to ^21.0.1 (#31274)
<!-- Please submit this PR as a draft initially. Do not mark it as "Ready for review" until the template has been completely filled out, and PR status checks have passed at least once. --> <!-- Write a short description of the changes included in this pull request, also include relevant motivation and context. Have in mind the following questions: 1. What is the reason for the change? 2. What is the improvement/solution? --> Bumping `@metamask/keyring-controller` to ^21.0.1 ```markdown - Fixed duplication of unsupported keyrings ([#5535](MetaMask/core#5535)) - Enforce keyrings metadata alignment when unlocking existing vault ([#5535](MetaMask/core#5535)) - Fixed frozen object mutation attempt when updating metadata ([#5535](MetaMask/core#5535)) ``` [![Open in GitHub Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/31293?quickstart=1) Fixes: #31200 1. Start with an installation from before we introduced keyringsMetadata, proceed past onboarding so that you have a vault setup (e.g. v12.12.0) 2. Upgrade to this branch 3. Start the extension (without logging in) 4. Leave the window open a while to give it time to initialize all controllers 5. Reload the extension before unlocking 6. Try to unlock the wallet <!-- If applicable, add screenshots and/or recordings to visualize the before and after of your change. --> <!-- [screenshots/recordings] --> <!-- [screenshots/recordings] --> - [ ] I've followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Extension Coding Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md). - [ ] I've completed the PR template to the best of my ability - [ ] I’ve included tests if applicable - [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [ ] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. - [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed). - [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.
1 parent c10cccc commit 0cef4aa

File tree

6 files changed

+10
-6
lines changed

6 files changed

+10
-6
lines changed

lavamoat/browserify/beta/policy.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1601,6 +1601,7 @@
16011601
},
16021602
"@metamask/keyring-controller": {
16031603
"globals": {
1604+
"console.error": true,
16041605
"console.log": true
16051606
},
16061607
"packages": {

lavamoat/browserify/flask/policy.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1601,6 +1601,7 @@
16011601
},
16021602
"@metamask/keyring-controller": {
16031603
"globals": {
1604+
"console.error": true,
16041605
"console.log": true
16051606
},
16061607
"packages": {

lavamoat/browserify/main/policy.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1601,6 +1601,7 @@
16011601
},
16021602
"@metamask/keyring-controller": {
16031603
"globals": {
1604+
"console.error": true,
16041605
"console.log": true
16051606
},
16061607
"packages": {

lavamoat/browserify/mmi/policy.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1732,6 +1732,7 @@
17321732
},
17331733
"@metamask/keyring-controller": {
17341734
"globals": {
1735+
"console.error": true,
17351736
"console.log": true
17361737
},
17371738
"packages": {

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -285,7 +285,7 @@
285285
"@metamask/json-rpc-engine": "^10.0.0",
286286
"@metamask/json-rpc-middleware-stream": "^8.0.4",
287287
"@metamask/keyring-api": "^17.2.1",
288-
"@metamask/keyring-controller": "^21.0.0",
288+
"@metamask/keyring-controller": "^21.0.1",
289289
"@metamask/keyring-internal-api": "^6.0.0",
290290
"@metamask/keyring-snap-client": "^4.0.1",
291291
"@metamask/logging-controller": "^6.0.4",

yarn.lock

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5613,9 +5613,9 @@ __metadata:
56135613
languageName: node
56145614
linkType: hard
56155615

5616-
"@metamask/keyring-controller@npm:^21.0.0":
5617-
version: 21.0.0
5618-
resolution: "@metamask/keyring-controller@npm:21.0.0"
5616+
"@metamask/keyring-controller@npm:^21.0.1":
5617+
version: 21.0.1
5618+
resolution: "@metamask/keyring-controller@npm:21.0.1"
56195619
dependencies:
56205620
"@ethereumjs/util": "npm:^9.1.0"
56215621
"@keystonehq/metamask-airgapped-keyring": "npm:^0.14.1"
@@ -5631,7 +5631,7 @@ __metadata:
56315631
ethereumjs-wallet: "npm:^1.0.1"
56325632
immer: "npm:^9.0.6"
56335633
ulid: "npm:^2.3.0"
5634-
checksum: 10/aee572da64d757417397fee74a44be1c3f0ea3168c6880cb211d99d71b5e1b457d1684760b9d347820c0d90f0052783105e60697e9315ae257d3ae12c852190a
5634+
checksum: 10/9406eb60ab111a9896a4e2db0ef009fcdd5dbe6daac1b06b77561af60e7463fcfc139727e0891203c0c135f744ac7bc5900b2e6f0bfe8dc5a218689be43bf266
56355635
languageName: node
56365636
linkType: hard
56375637

@@ -27274,7 +27274,7 @@ __metadata:
2727427274
"@metamask/json-rpc-engine": "npm:^10.0.0"
2727527275
"@metamask/json-rpc-middleware-stream": "npm:^8.0.4"
2727627276
"@metamask/keyring-api": "npm:^17.2.1"
27277-
"@metamask/keyring-controller": "npm:^21.0.0"
27277+
"@metamask/keyring-controller": "npm:^21.0.1"
2727827278
"@metamask/keyring-internal-api": "npm:^6.0.0"
2727927279
"@metamask/keyring-snap-client": "npm:^4.0.1"
2728027280
"@metamask/logging-controller": "npm:^6.0.4"

0 commit comments

Comments
 (0)