nodejs-github-bot · GitHub

@nodejs-github-bot added the crypto

Issues and PRs related to the crypto subsystem.

label

Nov 21, 2019

@tniessen

@tniessen tniessen added the semver-major

PRs that contain breaking changes and should be released in the next major version.

label

Nov 21, 2019

@tniessen

tniessen marked this pull request as ready for review

November 21, 2019 15:37

bnoordhuis

devnexen

RFC 2898 does not permit an iteration count of zero, and OpenSSL 1.1.1
will treat it as one iteration internally.
Future OpenSSL versions will reject such inputs (already on master
branch), but until that happens, Node.js should manually reject them.
Refs: nodejs/webcrypto#29

@tniessen

addaleax

addaleax pushed a commit that referenced this pull request

Nov 27, 2019
RFC 2898 does not permit an iteration count of zero, and OpenSSL 1.1.1
will treat it as one iteration internally.
Future OpenSSL versions will reject such inputs (already on master
branch), but until that happens, Node.js should manually reject them.
Refs: nodejs/webcrypto#29
PR-URL: #30578
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

@tniessen

tniessen deleted the crypto-forbid-iteration-count-zero branch

January 23, 2020 03:48

Merged

Read the original on github.com ↗