fix: default NODE_LOG_DIR will never use pm2_env.NODE_LOG_DIR

peze · peze · commit 8ad124471271 · 2019-11-12T19:25:28.000+08:00
Refs: #25
diff --git a/get_process_env.js b/get_process_env.js
@@ -44,7 +44,7 @@ function getProcessEnv(pid) {
   });
   const lines = output.split('\u0000');
 
-  var env = {ENABLE_NODE_LOG: '', NODE_LOG_DIR: '/tmp'};
+  var env = { ENABLE_NODE_LOG: '', NODE_LOG_DIR: '' };
   var pm2_env = {};
 
   for (var i = 0; i < lines.length; i++) {
@@ -65,7 +65,7 @@ function getProcessEnv(pid) {
     }
   }
 
-  env.NODE_LOG_DIR = env.NODE_LOG_DIR || pm2_env.NODE_LOG_DIR;
+  env.NODE_LOG_DIR = env.NODE_LOG_DIR || pm2_env.NODE_LOG_DIR || '/tmp';
   env.ENABLE_NODE_LOG = env.ENABLE_NODE_LOG || pm2_env.ENABLE_NODE_LOG;
 
   if (!env.NODE_LOG_DIR.endsWith('/')) {

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ function getProcessEnv(pid) {`
`44`	`44`	`});`
`45`	`45`	`const lines = output.split('\u0000');`
`46`	`46`
`47`		`- var env = {ENABLE_NODE_LOG: '', NODE_LOG_DIR: '/tmp'};`
	`47`	`+ var env = { ENABLE_NODE_LOG: '', NODE_LOG_DIR: '' };`
`48`	`48`	`var pm2_env = {};`
`49`	`49`
`50`	`50`	`for (var i = 0; i < lines.length; i++) {`
`@@ -65,7 +65,7 @@ function getProcessEnv(pid) {`
`65`	`65`	`}`
`66`	`66`	`}`
`67`	`67`
`68`		`- env.NODE_LOG_DIR = env.NODE_LOG_DIR \|\| pm2_env.NODE_LOG_DIR;`
	`68`	`+ env.NODE_LOG_DIR = env.NODE_LOG_DIR \|\| pm2_env.NODE_LOG_DIR \|\| '/tmp';`
`69`	`69`	`env.ENABLE_NODE_LOG = env.ENABLE_NODE_LOG \|\| pm2_env.ENABLE_NODE_LOG;`
`70`	`70`
`71`	`71`	`if (!env.NODE_LOG_DIR.endsWith('/')) {`