Skip to content

Commit

Permalink
Add ignore of stderr when executing process commands
Browse files Browse the repository at this point in the history
- Make sure any potential errors don't leak to the user
  • Loading branch information
ianmcnally committed Nov 22, 2016
1 parent e25f113 commit 03122b5
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion packages/react-dev-utils/getProcessForPort.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,14 @@ var chalk = require('chalk');
var execSync = require('child_process').execSync;
var path = require('path');

var execOptions = { encoding: 'utf8' };
var execOptions = {
encoding: 'utf8',
stdio: [
'pipe', // stdin (default)
'pipe', // stdout (default)
'ignore' //stderr
]
};

function isProcessAReactApp(processCommand) {
return /^node .*react-scripts\/scripts\/start\.js\s?$/.test(processCommand);
Expand Down

0 comments on commit 03122b5

Please sign in to comment.