fix(windows): fallback if native modules cannot be complied

Author: gergelyke

lib/agent/index.js

@@ -1,5 +1,5 @@
 var debug = require('debug')('risingstack/trace')
-var microtime = require('microtime')
+var microtime = require('../optionalDependencies/microtime')
 var uuid = require('node-uuid')
 var cls = require('@risingstack/continuation-local-storage')
 var defaults = require('lodash.defaults')

lib/agent/metrics/apm/index.js

@@ -1,6 +1,6 @@
 var os = require('os')
-var gc = require('gc-stats')()
-var eventLoopStats = require('event-loop-stats')
+var gc = require('../../../optionalDependencies/gc-stats')()
+var eventLoopStats = require('../../../optionalDependencies/event-loop-stats')
 
 var BYTES_TO_MEGABYTES = 1024 * 1024
 
@@ -41,13 +41,24 @@ function ApmMetrics (options) {
 }
 
 ApmMetrics.prototype.sendMetrics = function () {
-  this.collectorApi.sendApmMetrics({
+  var eventloop = this.getEventLoop()
+  var gc = this.getGC()
+
+  var databag = {
     timestamp: (new Date()).toISOString(),
     memory: this.getMemory(),
-    cpu: this.getCpu(),
-    eventloop: this.getEventLoop(),
-    gc: this.getGC()
-  })
+    cpu: this.getCpu()
+  }
+
+  if (eventloop.stats) {
+    databag.eventloop = eventloop
+  }
+
+  if (gc) {
+    databag.gc = gc
+  }
+
+  this.collectorApi.sendApmMetrics(databag)
 
   this.reset()
 }

lib/instrumentations/core/http/request.js

@@ -1,6 +1,6 @@
 var debug = require('debug')('risingstack/trace')
 var url = require('url')
-var microtime = require('microtime')
+var microtime = require('../../../optionalDependencies/microtime')
 
 var util = require('./util')
 var consts = require('../../../consts')

lib/instrumentations/core/http/server.js

@@ -1,7 +1,7 @@
 var url = require('url')
 
 var qs = require('qs')
-var microtime = require('microtime')
+var microtime = require('../../../optionalDependencies/microtime')
 var isNumber = require('lodash.isnumber')
 var debug = require('debug')('risingstack/trace')
 

lib/optionalDependencies/event-loop-stats.js

@@ -0,0 +1,12 @@
+var eventLoopStats
+
+try {
+  eventLoopStats = require('event-loop-stats')
+} catch (ex) {
+  console.log('event-loop-stats couldn\'t be required, possibly a compiler issue - continuing')
+  eventLoopStats = {
+    sense: function () {}
+  }
+}
+
+module.exports = eventLoopStats

lib/optionalDependencies/gc-stats.js

@@ -0,0 +1,14 @@
+var gcStats
+
+try {
+  gcStats = require('gc-stats')
+} catch (ex) {
+  gcStats = function () {
+    console.log('gc-stats couldn\'t be required, possibly a compiler issue - continuing')
+    return {
+      on: function () {}
+    }
+  }
+}
+
+module.exports = gcStats

lib/optionalDependencies/microtime.js

@@ -0,0 +1,14 @@
+var microtime
+
+try {
+  microtime = require('microtime')
+} catch (ex) {
+  console.log('microtime couldn\'t be required, possibly a compiler issue - continuing')
+  microtime = {
+    now: function () {
+      return Date.now() * 1000
+    }
+  }
+}
+
+module.exports = microtime

package.json

@@ -43,16 +43,18 @@
     "@risingstack/continuation-local-storage": "1.0.2",
     "bl": "1.1.2",
     "debug": "2.2.0",
-    "event-loop-stats": "1.0.0",
-    "gc-stats": "1.0.0",
     "lodash.assign": "4.0.6",
     "lodash.defaults": "4.0.1",
     "lodash.isnumber": "3.0.3",
-    "microtime": "2.0.0",
     "node-uuid": "1.4.7",
     "qs": "6.1.0",
     "sync-request": "3.0.0"
   },
+  "optionalDependencies": {
+    "microtime": "2.0.0",
+    "event-loop-stats": "1.0.0",
+    "gc-stats": "1.0.0"
+  },
   "devDependencies": {
     "amqplib": "0.4.1",
     "async": "1.5.2",