Merge pull request #43 from capaj/unref

RafalWilinski · web-flow · commit f2edcb09d37f · 2016-09-01T19:36:06.000+01:00
unref interval
diff --git a/index.js b/index.js
@@ -31,7 +31,8 @@ const middlewareWrapper = (config) => {
       config.spans.forEach((span) => {
         span.os = [];
         span.responses = [];
-        setInterval(() => gatherOsMetrics(io, span), span.interval * 1000);
+        const interval = setInterval(() => gatherOsMetrics(io, span), span.interval * 1000);
+        interval.unref(); // don't keep node.js process up
       });
     }
 
diff --git a/package.json b/package.json
@@ -10,6 +10,9 @@
     "express",
     "charts"
   ],
+  "engines" : {
+    "node" : ">=4"
+  },
   "author": "Rafal Wilinski <raf.wilinski@gmail.com> (http://rwilinski.me)",
   "contributors": [
     {

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,8 @@ const middlewareWrapper = (config) => {`
`31`	`31`	`config.spans.forEach((span) => {`
`32`	`32`	`span.os = [];`
`33`	`33`	`span.responses = [];`
`34`		`- setInterval(() => gatherOsMetrics(io, span), span.interval * 1000);`
	`34`	`+ const interval = setInterval(() => gatherOsMetrics(io, span), span.interval * 1000);`
	`35`	`+ interval.unref(); // don't keep node.js process up`
`35`	`36`	`});`
`36`	`37`	`}`
`37`	`38`