build,lib,test: change whitelist to allowlist

PR-URL: #36406 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Dec 21, 2020 · 1330995 · 1330995
1 parent 129053f
commit 1330995
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 12 deletions.
diff --git a/configure.py b/configure.py
@@ -948,7 +948,7 @@ def get_gas_version(cc):
 
 # Note: Apple clang self-reports as clang 4.2.0 and gcc 4.2.1.  It passes
 # the version check more by accident than anything else but a more rigorous
-# check involves checking the build number against a whitelist.  I'm not
+# check involves checking the build number against an allowlist.  I'm not
 # quite prepared to go that far yet.
 def check_compiler(o):
   if sys.platform == 'win32':

diff --git a/lib/internal/bootstrap/loaders.js b/lib/internal/bootstrap/loaders.js
@@ -69,11 +69,11 @@ ObjectDefineProperty(process, 'moduleLoadList', {
 });
 
 
-// internalBindingWhitelist contains the name of internalBinding modules
-// that are whitelisted for access via process.binding()... This is used
+// internalBindingAllowlist contains the name of internalBinding modules
+// that are allowed for access via process.binding()... This is used
 // to provide a transition path for modules that are being moved over to
 // internalBinding.
-const internalBindingWhitelist = new SafeSet([
+const internalBindingAllowlist = new SafeSet([
   'async_wrap',
   'buffer',
   'cares_wrap',
@@ -113,7 +113,7 @@ const internalBindingWhitelist = new SafeSet([
     module = String(module);
     // Deprecated specific process.binding() modules, but not all, allow
     // selective fallback to internalBinding for the deprecated ones.
-    if (internalBindingWhitelist.has(module)) {
+    if (internalBindingAllowlist.has(module)) {
       return internalBinding(module);
     }
     // eslint-disable-next-line no-restricted-syntax

diff --git a/test/common/README.md b/test/common/README.md
@@ -40,12 +40,12 @@ The `benchmark` module is used by tests to run benchmarks.
 The `common` module is used by tests for consistency across repeated
 tasks.
 
-### `allowGlobals(...whitelist)`
+### `allowGlobals(...allowlist)`
 
-* `whitelist` [&lt;Array>][] Array of Globals
+* `allowlist` [&lt;Array>][] Array of Globals
 * return [&lt;Array>][]
 
-Takes `whitelist` and concats that with predefined `knownGlobals`.
+Takes `allowlist` and concats that with predefined `knownGlobals`.
 
 ### `canCreateSymLink()`
 
@@ -601,7 +601,7 @@ If set, crypto tests are skipped.
 ### `NODE_TEST_KNOWN_GLOBALS`
 
 A comma-separated list of variables names that are appended to the global
-variable whitelist. Alternatively, if `NODE_TEST_KNOWN_GLOBALS` is set to `'0'`,
+variable allowlist. Alternatively, if `NODE_TEST_KNOWN_GLOBALS` is set to `'0'`,
 global leak detection is disabled.
 
 ## Fixtures Module

diff --git a/test/common/index.js b/test/common/index.js
@@ -276,8 +276,8 @@ if (global.gc) {
   knownGlobals.push(global.gc);
 }
 
-function allowGlobals(...whitelist) {
-  knownGlobals = knownGlobals.concat(whitelist);
+function allowGlobals(...allowlist) {
+  knownGlobals = knownGlobals.concat(allowlist);
 }
 
 if (process.env.NODE_TEST_KNOWN_GLOBALS !== '0') {

diff --git a/...cess-binding-internalbinding-whitelist.js → ...cess-binding-internalbinding-allowlist.js b/...cess-binding-internalbinding-whitelist.js → ...cess-binding-internalbinding-allowlist.js
@@ -4,7 +4,7 @@
 const common = require('../common');
 const assert = require('assert');
 
-// Assert that whitelisted internalBinding modules are accessible via
+// Assert that allowed internalBinding modules are accessible via
 // process.binding().
 assert(process.binding('async_wrap'));
 assert(process.binding('buffer'));