-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
feat(ext/crypto): implement deriveKey #12117
Conversation
Please rebase |
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. Thanks.
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 too
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 but has merge conflicts (in expectation.json presumably.)
ext/crypto/00_crypto.js
Outdated
); | ||
} | ||
} else if (algorithm.length !== 0) { | ||
length == algorithm.length; |
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.
length == algorithm.length; | |
length = algorithm.length; |
I'd just return
here (and above too - easier to reason about.)
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.
It's just following the steps as per spec. Easier for anyone reading/debugging with the spec open alongside IMO
Towards #11690
Large WPT diff because of missing importKey and some previously false tests were disabled.