vm: deprecate vm.runInDebugContext

nodejs · Oct 22, 2017 · 66fa03d · 66fa03d
1 parent 438e7fd
commit 66fa03d
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 1 deletion.
diff --git a/doc/api/vm.md b/doc/api/vm.md
@@ -313,6 +313,11 @@ console.log(util.inspect(sandbox));
 ## vm.runInDebugContext(code)
 <!-- YAML
 added: v0.11.14
+deprecated: v8.0.0
+changes:
+    - version: REPLACEME
+      pr-url: https://github.com/nodejs/node/pull/12815
+      description: Calling this function now emits a deprecation warning.
 -->
 
 > Stability: 0 - Deprecated. An alternative is in development.

diff --git a/lib/util.js b/lib/util.js
@@ -385,7 +385,11 @@ function stylizeNoColor(str, styleType) {
 function ensureDebugIsInitialized() {
   if (Debug === undefined) {
     const runInDebugContext = require('vm').runInDebugContext;
+    // a workaround till this entire method is removed
+    const originalValue = process.noDeprecation;
+    process.noDeprecation = true;
     Debug = runInDebugContext('Debug');
+    process.noDeprecation = originalValue;
   }
 }
 

diff --git a/lib/vm.js b/lib/vm.js
@@ -27,7 +27,7 @@ const {
 
   makeContext,
   isContext,
-  runInDebugContext
+  runInDebugContext: runInDebugContext_
 } = process.binding('contextify');
 
 // The binding provides a few useful primitives:
@@ -105,6 +105,19 @@ function sigintHandlersWrap(fn, thisArg, argsArray) {
   }
 }
 
+let runInDebugContextWarned = false;
+function runInDebugContext(code) {
+  if (runInDebugContextWarned === false) {
+    runInDebugContextWarned = true;
+    process.emitWarning(
+      'DebugContext has been deprecated and will be removed in a ' +
+      'future version.',
+      'DeprecationWarning',
+      'DEP0069');
+  }
+  return runInDebugContext_(code);
+}
+
 function runInContext(code, contextifiedSandbox, options) {
   if (typeof options === 'string') {
     options = {

diff --git a/test/parallel/test-vm-debug-context.js b/test/parallel/test-vm-debug-context.js
@@ -27,6 +27,11 @@ const vm = require('vm');
 const { spawn } = require('child_process');
 const fixtures = require('../common/fixtures');
 
+const msg = 'DebugContext has been deprecated and will be removed in ' +
+            'a future version.';
+common.expectWarning('DeprecationWarning', msg);
+vm.runInDebugContext();
+
 assert.throws(function() {
   vm.runInDebugContext('*');
 }, /SyntaxError/);