module: check -e flag in debug break setup

When both --debug-brk and --eval are set, and a filename is specified, its full path is not set correctly, causing an error for relative filenames with './' omitted. For example, 'node --debug-brk -e 0 hello.js' throws an error. Since the script referenced by the filename is never run anyway, this change skips resolving its full path if both --debug-brk and --eval are set.
nodejs · Oct 1, 2016 · 9f22b8c · 9f22b8c
1 parent f5d997c
commit 9f22b8c
Show file tree

Hide file tree

Showing 2 changed files with 44 additions and 29 deletions.
diff --git a/lib/module.js b/lib/module.js
@@ -549,7 +549,7 @@ Module.prototype._compile = function(content, filename) {
     displayErrors: true
   });
 
-  if (process._debugWaitConnect) {
+  if (process._debugWaitConnect && process._eval == null) {
     if (!resolvedArgv) {
       // we enter the repl if we're not given a filename argument.
       if (process.argv[1]) {

diff --git a/test/parallel/test-debug-brk.js b/test/parallel/test-debug-brk.js
@@ -3,33 +3,48 @@
 const common = require('../common');
 const spawn = require('child_process').spawn;
 
-var procStderr = '';
-var agentStdout = '';
-var needToSpawnAgent = true;
-var needToExit = true;
-
-const procArgs = [`--debug-brk=${common.PORT}`, '-e', '0'];
-const proc = spawn(process.execPath, procArgs);
-proc.stderr.setEncoding('utf8');
-
-const exitAll = common.mustCall((processes) => {
-  processes.forEach((myProcess) => { myProcess.kill(); });
-});
-
-proc.stderr.on('data', (chunk) => {
-  procStderr += chunk;
-  if (/Debugger listening on/.test(procStderr) && needToSpawnAgent) {
-    needToSpawnAgent = false;
-    const agentArgs = ['debug', `localhost:${common.PORT}`];
-    const agent = spawn(process.execPath, agentArgs);
-    agent.stdout.setEncoding('utf8');
-
-    agent.stdout.on('data', (chunk) => {
-      agentStdout += chunk;
-      if (/connecting to .+ ok/.test(agentStdout) && needToExit) {
-        needToExit = false;
-        exitAll([proc, agent]);
+let run = () => {};
+function test(extraArgs) {
+  const next = run;
+  run = () => {
+    var procStderr = '';
+    var agentStdout = '';
+    var needToSpawnAgent = true;
+    var needToExit = true;
+
+    const procArgs = [`--debug-brk=${common.PORT}`].concat(extraArgs);
+    const proc = spawn(process.execPath, procArgs);
+    proc.stderr.setEncoding('utf8');
+
+    const exitAll = common.mustCall((processes) => {
+      processes.forEach((myProcess) => { myProcess.kill(); });
+    });
+
+    proc.stderr.on('data', (chunk) => {
+      procStderr += chunk;
+      if (/Debugger listening on/.test(procStderr) && needToSpawnAgent) {
+        needToSpawnAgent = false;
+        const agentArgs = ['debug', `localhost:${common.PORT}`];
+        const agent = spawn(process.execPath, agentArgs);
+        agent.stdout.setEncoding('utf8');
+
+        agent.stdout.on('data', (chunk) => {
+          agentStdout += chunk;
+          if (/connecting to .+ ok/.test(agentStdout) && needToExit) {
+            needToExit = false;
+            exitAll([proc, agent]);
+          }
+        });
       }
     });
-  }
-});
+
+    proc.on('exit', () => {
+      next();
+    });
+  };
+}
+
+test(['-e', '0']);
+test(['-e', '0', 'foo']);
+
+run();