Skip to content

crypto: upgrade pbkdf2 without digest to an error

Rodrigo Muino Tomonari requested to merge github/fork/jasnell/crypto-error-pbkdf2-without-digest into master

Commit a1163582 added a deprecation warning when pbkdf2 was called without an explicit digest argument. This was because the default digest is sha1, which is not-recommended from a security point of view. This upgrades it to a runtime error when digest is undefined per the plan discussed in the original issue.

Ref: https://github.com/nodejs/node/commit/a1163582c53dc6e00f3680084269600913b1cad2

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

crypto

Merge request reports