Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement node crypto algorithm listings, crypto error message fixes #856

Merged
merged 2 commits into from
Jul 13, 2023

Conversation

fhanau
Copy link
Collaborator

@fhanau fhanau commented Jul 11, 2023

See getHashes() for the rationale for hardcoding getCiphers() and getCurves(). The second commit fixes several error messages, see the commit description.

fhanau added 2 commits July 11, 2023 16:26
Previously, WebCrypto deriveBits() error messages would list the private key type twice if there was a type mismatch. Node crypto's createDiffieHellmanGroup() would indicate that only group modp5 is supported whereas the actual supported groups are modp14 to modp18.
@fhanau fhanau requested review from jasnell and Warfields July 11, 2023 20:36
@fhanau fhanau merged commit 967f719 into main Jul 13, 2023
@fhanau fhanau deleted the felix/node-crypto-createCC branch July 13, 2023 01:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants