Skip to content

[v9.x backport] crypto: add ECDH.convertKey to convert public keys - #19745

Closed
targos wants to merge 2 commits into
nodejs:v9.x-stagingfrom
targos:backport-ecdh-convert-key
Closed

[v9.x backport] crypto: add ECDH.convertKey to convert public keys#19745
targos wants to merge 2 commits into
nodejs:v9.x-stagingfrom
targos:backport-ecdh-convert-key

Conversation

@targos

Copy link
Copy Markdown
Member

ECDH.convertKey is used to convert public keys between different
formats.

PR-URL: #19080
Fixes: #18977
Reviewed-By: Ben Noordhuis info@bnoordhuis.nl
Reviewed-By: James M Snell jasnell@gmail.com

/cc @tniessen

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. v9.x labels Apr 2, 2018
@targos

Copy link
Copy Markdown
MemberAuthor

Comment threaddoc/api/crypto.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit:

-- `format` {string} Defaults to `uncompressed`.+- `format` {string} **Default:** `uncompressed`

As per #19737 (will be backported, but without this fix).

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Updated.

ECDH.convertKey is used to convert public keys between different
formats.
PR-URL: nodejs#19080Fixes: nodejs#18977
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos
targosforce-pushed the backport-ecdh-convert-key branch from 73b22ed to 1b408f8CompareApril 4, 2018 13:07
@targos
targosforce-pushed the backport-ecdh-convert-key branch from 1b408f8 to 49573b7CompareApril 4, 2018 13:08
@targos

Copy link
Copy Markdown
MemberAuthor

targos pushed a commit that referenced this pull request Apr 12, 2018
ECDH.convertKey is used to convert public keys between different
formats.
PR-URL: #19080Fixes: #18977
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Backport-PR-URL: #19745
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@targos

Copy link
Copy Markdown
MemberAuthor

Landed in f06b41e

@targostargos closed this Apr 12, 2018
@targos
targos deleted the backport-ecdh-convert-key branch April 12, 2018 13:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.lib / srcIssues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@targos@tniessen@vsemozhetbyt@nodejs-github-bot@wuweiweiwu