diff --git a/lib/internal/bootstrap_node.js b/lib/internal/bootstrap_node.js index 9755641b0de4b1..62b6911cbf9062 100644 --- a/lib/internal/bootstrap_node.js +++ b/lib/internal/bootstrap_node.js @@ -327,10 +327,10 @@ } function setupInspector(originalConsole, wrappedConsole, Module) { - const { addCommandLineAPI, consoleCall } = process.binding('inspector'); - if (!consoleCall) { + if (!process.config.variables.v8_enable_inspector) { return; } + const { addCommandLineAPI, consoleCall } = process.binding('inspector'); // Setup inspector command line API const { makeRequireFunction } = NativeModule.require('internal/module'); const path = NativeModule.require('path'); diff --git a/test/fixtures/overwrite-config-preload-module.js b/test/fixtures/overwrite-config-preload-module.js new file mode 100644 index 00000000000000..21cb966a54988b --- /dev/null +++ b/test/fixtures/overwrite-config-preload-module.js @@ -0,0 +1,5 @@ +'use strict' +const common = require('../common'); +common.skipIfInspectorDisabled(); + +process.config = {}; diff --git a/test/sequential/test-inspector-overwrite-config.js b/test/sequential/test-inspector-overwrite-config.js new file mode 100644 index 00000000000000..ecb14c6638edb5 --- /dev/null +++ b/test/sequential/test-inspector-overwrite-config.js @@ -0,0 +1,41 @@ +// Flags: --require ./test/fixtures/overwrite-config-preload-module.js +'use strict'; + +// This test ensures that overwriting a process configuration +// value does not affect code in bootstrap_node.js. Specifically this tests +// that the inspector console functions are bound even though +// overwrite-config-preload-module.js overwrote the process.config variable. + +// We cannot do a check for the inspector because the configuration variables +// were reset/removed by overwrite-config-preload-module.js. +/* eslint-disable inspector-check */ + +const common = require('../common'); +const assert = require('assert'); +const inspector = require('inspector'); +const msg = 'Test inspector logging'; +let asserted = false; + +async function testConsoleLog() { + const session = new inspector.Session(); + session.connect(); + session.on('inspectorNotification', (data) => { + if (data.method === 'Runtime.consoleAPICalled') { + assert.strictEqual(data.params.args.length, 1); + assert.strictEqual(data.params.args[0].value, msg); + asserted = true; + } + }); + session.post('Runtime.enable'); + console.log(msg); + session.disconnect(); +} + +common.crashOnUnhandledRejection(); + +async function runTests() { + await testConsoleLog(); + assert.ok(asserted, 'log statement did not reach the inspector'); +} + +runTests();