refactor: only enable deep monitoring if explicitly stated

Unitech · Aug 23, 2018 · f67e14f · f67e14f
1 parent 95321c6
commit f67e14f
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/lib/ProcessUtils.js b/lib/ProcessUtils.js
@@ -12,27 +12,28 @@ module.exports = {
         conf = io.conf ? io.conf : conf;
       }
 
-      var activate = process.env.km_link == 'true' || process.env.deep_monitoring === 'true' || false;
+      var activate_lvl1 = process.env.km_link == 'true' || false;
+      var activate_lvl2 = process.env.deep_monitoring === 'true';
 
       let defaultConf = {
-        transactions: (process.env.trace === 'true' || process.env.deep_monitoring === 'true') || false,
-        http: activate,
+        transactions: process.env.trace === 'true' || false,
+        http: activate_lvl1,
         metrics: {
-          deepMetrics: activate,
-          v8: activate || process.env.v8 === 'true'
+          deepMetrics: activate_lvl2,
+          v8: activate_lvl2 || process.env.v8 === 'true'
         },
         actions: {
-          eventLoopDump: activate,
-          profilingCpu: activate,
-          profilingHeap: activate
+          eventLoopDump: activate_lvl1,
+          profilingCpu: activate_lvl1,
+          profilingHeap: activate_lvl1
         }
       };
 
       const mergedConf = Object.assign(defaultConf, conf);
 
       pmx.init(mergedConf);
 
-      if (activate && require('semver').satisfies(process.versions.node, '>= 8.0.0')) {
+      if (activate_lvl1 && require('semver').satisfies(process.versions.node, '>= 8.0.0')) {
         var url = '';
         pmx.action('internal:inspect', function(reply) {
           const inspector = require('inspector');