| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Before const crypto = require('crypto');
const assert = require('assert');
const data = Buffer.from('hello');
const { privateKey } = crypto.generateKeyPairSync('ed25519');
const signature = crypto.sign(null, data, privateKey);
assert(crypto.verify(null, data, privateKey, signature)); // OK
crypto.verify(null, data, privateKey, signature, (err, verified) => { // 💥
assert(!err);
assert(verified);
});
// node[49326]: ../src/crypto/crypto_sig.cc:850:static bool node::crypto::SignTraits::DeriveBits(node::Environment *, const node::crypto::SignConfiguration &, node::crypto::ByteSource *): Assertion `(params.key->GetKeyType()) == (kKeyTypePublic)' failed.After const crypto = require('crypto');
const assert = require('assert');
const data = Buffer.from('hello');
const { privateKey } = crypto.generateKeyPairSync('ed25519');
const signature = crypto.sign(null, data, privateKey);
assert(crypto.verify(null, data, privateKey, signature)); // OK
crypto.verify(null, data, privateKey, signature, (err, verified) => {
assert(!err);
assert(verified); // OK
}); |
Sorry, something went wrong.
Sorry, something went wrong.
|
cc @nodejs/crypto |
Sorry, something went wrong.
|
@jasnell should we fast-track and do v15.12.1? |
Sorry, something went wrong.
|
Yeah i think so |
Sorry, something went wrong.
|
👍 to fast-track |
Sorry, something went wrong.
Sorry, something went wrong.
|
@nodejs/releasers This fixes a possible crash introduced in a new optional argument added to a stable API with yesterday's 15.12.0 release (#37500). We're proposing a quick 15.12.1 patch release. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
fixes #37794