Skip to content

Commit

Permalink
lib: runtime deprecate access to process.binding('v8')
Browse files Browse the repository at this point in the history
The only useful functions are already exposed via public APIs.
There's no reason for user code to access the binding.

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #37789
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
jasnell committed Mar 22, 2021
1 parent 42ff00e commit ae595d7
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lib/internal/bootstrap/loaders.js
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,7 @@ const internalBindingAllowlist = new SafeSet([

const runtimeDeprecatedList = new SafeSet([
'async_wrap',
'v8',
]);

// Set up process.binding() and process._linkedBinding().
Expand Down

0 comments on commit ae595d7

Please sign in to comment.