Skip to content

tools: add check for using process.binding crypto

Rodrigo Muino Tomonari requested to merge github/fork/danbev/eslint-crypto-binding-check into master

Currently, when configuring --without-ssl any tests that use process.binding('crypto') will not report a lint warning. This is because the eslint check only generates a warning when using require.

This commit adds a check for using binding in addition to require.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

tools

Merge request reports