Project

General

Profile

Task #4764

Use Node.js Crypto keygen

Added by Jeff Thompson 9 months ago. Updated about 1 month ago.

Status:
New
Priority:
Normal
Assignee:
Start date:
10/29/2018
Due date:
% Done:

0%

Estimated time:

Description

As of October 10, the Node.js 10.12 Crypto library (finally) supports RSA key generation. When the package managers for our supported platforms update to Node.js 10.12, and NDN-JS users agree, we should stop using the npm package rsa-keygen and change to the Crypto generateKeyPairSync.

https://nodejs.org/api/crypto.html#crypto_crypto_generatekeypairsync_type_options

History

#1 Updated by Jeff Thompson 7 months ago

Thanks to a pull request from Junxiao, we switched to rsa-keypair. But we are still waiting for supported platforms to update to Node.js 10.12.

https://github.com/named-data/ndn-js/pull/71

#2 Updated by Junxiao Shi about 1 month ago

rsa-keypair is broken in NodeJS 12.x.
Can you define what are "supported platforms"? Post answer in #4809

#3 Updated by Jeff Thompson about 1 month ago

It looks like brew on macOS installs Node 12.x. But on Ubuntu 16.04, apt-get installs Node 4.x, and on Ubuntu 18.04 it installs Node 8.x. So, we can't drop support for versions before 12. I suppose we need to do a version check in the code.

Also available in: Atom PDF