Merge pull request #32 from signalfx/fix2ndReconnectStart

fix issue where the 2nd patch mode reconnect would have an incorrect …

Author: kchengsf

lib/client/signalflow/request_manager.js

@@ -213,14 +213,18 @@ function RequestManager(endPoint) {
   function addComputation(params, requestId, transportType, cb, requestType, isRetryPatchMode) {
     cb = cb || {};
     var router = routedMessageHandler(params, cb.onMessage, cb.onError, isRetryPatchMode);
+    var range = params.range;
+    if (!range) {
+      range = (params.end ? params.end : Date.now()) - params.start;
+    }
     knownComputations[requestId] = {
       params: params,
       callback: cb,
       onMessage: router.onMessage,
       transport: transportType,
       requestId: requestId,
       lastDataBatchTime: router.getLatestBatchTimeStamp,
-      range: Date.now() - params.start,
+      range: range,
       requestType: requestType, // unused at the moment but useful for debugging
       isRetryPatchMode: isRetryPatchMode
     };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "signalfx",
-  "version": "4.0.23",
+  "version": "4.0.24",
   "description": "Node.js client library for SignalFx",
   "homepage": "https://signalfx.com",
   "repository": "https://github.com/signalfx/signalfx-nodejs",