Stripping environment variables from the HttpInterface (#2895)

* Fixes 452

* Renamed to `proc`

* Default to false to keep backwards compatibility
This commit is contained in:
Michiel De Mey 2017-05-23 23:48:21 +02:00 committed by vmarchaud
parent 22df02093e
commit 9433f1fc16
2 changed files with 12 additions and 0 deletions

View File

@ -80,6 +80,7 @@ var csts = {
DEBUG : process.env.PM2_DEBUG || false,
WEB_IPADDR : process.env.PM2_API_IPADDR || '0.0.0.0',
WEB_PORT : parseInt(process.env.PM2_API_PORT) || 9615,
WEB_STRIP_ENV_VARS : process.env.PM2_WEB_STRIP_ENV_VARS || false,
MODIFY_REQUIRE : process.env.PM2_MODIFY_REQUIRE || false,
WORKER_INTERVAL : process.env.PM2_WORKER_INTERVAL || 30000,

View File

@ -46,6 +46,17 @@ function startWebServer(pm2) {
processes: list
};
if (cst.WEB_STRIP_ENV_VARS === true) {
for (var i = data.processes.length - 1; i >= 0; i--) {
var proc = data.processes[i];
// Strip important environment variables
if (typeof proc.pm2_env === 'undefined' && typeof proc.pm2_env.env === 'undefined') return;
delete proc.pm2_env.env;
}
}
res.statusCode = 200;
res.write(JSON.stringify(data));
return res.end();