-
-
Notifications
You must be signed in to change notification settings - Fork 234
Seedless controller: store keyring encryption key instead of password #5995
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
11e3287
to
e149187
Compare
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
01cb507
to
abf6aae
Compare
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
9399e4f
to
4e3fd4c
Compare
@metamaskbot publish-preview |
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
2dca1a5
to
3636a12
Compare
@metamaskbot publish-preview |
3636a12
to
a088347
Compare
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions.
|
a088347
to
35c6ef2
Compare
369af94
to
f2ef812
Compare
35c6ef2
to
efdfa8c
Compare
fee5652
to
9688e86
Compare
cd506c7
to
a72257b
Compare
9688e86
to
3b7ffdd
Compare
this is necessary to rehydrate vault after password change without knowledge of old password
This reverts commit b3b2e4c.
a72257b
to
b0f5338
Compare
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Show resolved
Hide resolved
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
b1e7582
to
9b7a0ed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Outdated
Show resolved
Hide resolved
packages/seedless-onboarding-controller/src/SeedlessOnboardingController.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Explanation
It has been discussed that the seedless controller should no longer store and recover the vault password, instead it should store and recover the vault encryption key for the purpose of vault recovery in case of a global password change.
Breaking
submitGlobalPassword
,storeKeyringEncryptionKey
, andloadKeyringEncryptionKey
.syncLatestGlobalPassword
.recoverCurrentDevicePassword
.References
Previously, seedless onboarding was backing up the keyring password to allow for vault recovery after a password change. Now we backup the keyring encryption key.
See the ADR for more details.
This is part of the implementation of option 6.
Changelog
Checklist