-
-
Notifications
You must be signed in to change notification settings - Fork 133
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
[15.x] Backport encryption improvements #312
Conversation
* fix: prefer encryptionKey for encryption when possible * refactor: add test case
* chore: update browser-passworder * refactor: remove `updateVault` from `GenericEncryptor`
@Gudahtt we probably want to merge this without squashing to preserve the cherry-picked commits, but it is currently not allowed by the repo settings |
Just as a heads up, I've renamed the branch from |
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!
I think we'll want to regular merge when syncing this back into the main branch, after the next v15 release goes out. Merging here wouldn't have an advantage because commits are already unique to this branch, i.e. they already won't line up with the same commits on main |
* [15.x] Backport encryption improvements (#312) * Prefer cached `encryptionKey` for encryption when possible (#307) * fix: prefer encryptionKey for encryption when possible * refactor: add test case * Use encryptor `isVaultUpdated` (#310) * chore: update browser-passworder * refactor: remove `updateVault` from `GenericEncryptor` * 15.1.0 (#313) * 15.1.0 * edit changelog entries * rephrase changelog entry --------- Co-authored-by: github-actions <github-actions@github.com> Co-authored-by: Michele Esposito <michele@esposito.codes> --------- Co-authored-by: Michele Esposito <34438276+mikesposito@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions <github-actions@github.com> Co-authored-by: Michele Esposito <michele@esposito.codes> Co-authored-by: legobeat <109787230+legobeat@users.noreply.github.com>
Description
This PR backports the following commits to
15.x
:Changes
isVaultUpdated
property toGenericEncryptor
type@metamask/browser-passworder
to^4.3.0
isVaultUpdated
method, the latter is used to check if the vault is updated instead ofupdateVault
GenericEncryptor
'supdateVault
property has been removed as no longer usedReferences
Checklist