fix: protect geteuid/getegid from being called on windows #3793

Unitech · Jul 17, 2018 · 0495bd8 · 0495bd8
1 parent 56f5e04
commit 0495bd8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/API/Extra.js b/lib/API/Extra.js
@@ -67,9 +67,9 @@ module.exports = function(CLI) {
         fmt.field('argv', process.argv);
         fmt.field('argv0', process.argv0);
         fmt.field('user', process.env.USER || process.env.LNAME || process.env.USERNAME);
-        if (process.geteuid)
+        if (cst.IS_WINDOWS === false && process.geteuid)
           fmt.field('uid', process.geteuid());
-        if (process.getegid)
+        if (cst.IS_WINDOWS === false && process.getegid)
           fmt.field('gid', process.getegid());
 
         var os = require('os');

diff --git a/lib/God/ActionMethods.js b/lib/God/ActionMethods.js
@@ -859,8 +859,8 @@ module.exports = function(God) {
       argv0 : process.argv0,
       argv : process.argv,
       user : process.env.USER,
-      uid : process.geteuid ? process.geteuid() : 'N/A',
-      gid : process.getegid ? process.getegid() : 'N/A',
+      uid : (cst.IS_WINDOWS === false && process.geteuid) ? process.geteuid() : 'N/A',
+      gid : (cst.IS_WINDOWS === false && process.getegid) ? process.getegid() : 'N/A',
       env : process.env,
       managed_apps : Object.keys(God.clusters_db).length,
       started_at : God.started_at