rollbar 2.25.1 → 2.26.0

package/index.d.ts

@@ -75,7 +75,7 @@ declare namespace Rollbar {
         hostBlockList?: string[];
         hostWhiteList?: string[]; // deprecated
         hostSafeList?: string[];
-        ignoredMessages?: string[];
+        ignoredMessages?: (string | RegExp)[];
         ignoreDuplicateErrors?: boolean;
         includeItemsInTelemetry?: boolean;
         inspectAnonymousErrors?: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rollbar",
-  "version": "2.25.1",
+  "version": "2.26.0",
   "repository": {
     "type": "git",
     "url": "http://github.com/rollbar/rollbar.js"

package/src/apiUtility.js

@@ -25,6 +25,7 @@ function getTransportFromOptions(options, defaults, url) {
   var path = defaults.path;
   var search = defaults.search;
   var timeout = options.timeout;
+  var transport = detectTransport(options)
 
   var proxy = options.proxy;
   if (options.endpoint) {
@@ -42,16 +43,26 @@ function getTransportFromOptions(options, defaults, url) {
     port: port,
     path: path,
     search: search,
-    proxy: proxy
+    proxy: proxy,
+    transport: transport
   };
 }
 
+function detectTransport(options) {
+  var gWindow = ((typeof window != 'undefined') && window) || ((typeof self != 'undefined') && self);
+  var transport = options.defaultTransport || 'xhr';
+  if (typeof gWindow.fetch === 'undefined') transport = 'xhr';
+  if (typeof gWindow.XMLHttpRequest === 'undefined') transport = 'fetch';
+  return transport;
+}
+
 function transportOptions(transport, method) {
   var protocol = transport.protocol || 'https:';
   var port = transport.port || (protocol === 'http:' ? 80 : protocol === 'https:' ? 443 : undefined);
   var hostname = transport.hostname;
   var path = transport.path;
   var timeout = transport.timeout;
+  var transportAPI = transport.transport;
   if (transport.search) {
     path = path + transport.search;
   }
@@ -67,7 +78,8 @@ function transportOptions(transport, method) {
     hostname: hostname,
     path: path,
     port: port,
-    method: method
+    method: method,
+    transport: transportAPI
   };
 }
 

package/src/browser/telemetry.js

@@ -1,4 +1,5 @@
 var _ = require('../utility');
+var headers = require('../utility/headers');
 var scrub = require('../scrub');
 var urlparser = require('./url');
 var domUtil = require('./domUtility');
@@ -362,7 +363,7 @@ Instrumenter.prototype.instrumentNetwork = function() {
         if (args[1] && args[1].headers) {
           // Argument may be a Headers object, or plain object. Ensure here that
           // we are working with a Headers object with case-insensitive keys.
-          var reqHeaders = new Headers(args[1].headers);
+          var reqHeaders = headers(args[1].headers);
 
           metadata.request_content_type = reqHeaders.get('Content-Type');
 
@@ -382,6 +383,9 @@ Instrumenter.prototype.instrumentNetwork = function() {
         if (self.trackHttpErrors()) {
           metadata.stack = (new Error()).stack;
         }
+
+        // Start our handler before returning the promise. This allows resp.clone()
+        // to execute before other handlers touch the response.
         return orig.apply(this, args).then(function (resp) {
           metadata.end_time_ms = _.now();
           metadata.status_code = resp.status;
@@ -394,6 +398,7 @@ Instrumenter.prototype.instrumentNetwork = function() {
           if (self.autoInstrument.networkResponseBody) {
             if (typeof resp.text === 'function') { // Response.text() is not implemented on some platforms
               // The response must be cloned to prevent reading (and locking) the original stream.
+              // This must be done before other handlers touch the response.
               body = resp.clone().text(); //returns a Promise
             }
           }

package/src/browser/transforms.js

@@ -79,20 +79,25 @@ function addBaseInfo(item, options, callback) {
 
 function addRequestInfo(window) {
   return function(item, options, callback) {
-    if (!window || !window.location) {
-      return callback(null, item);
+    var requestInfo = {};
+
+    if (window && window.location) {
+      requestInfo.url = window.location.href;
+      requestInfo.query_string = window.location.search;
     }
+
     var remoteString = '$remote_ip';
     if (!options.captureIp) {
       remoteString = null;
     } else if (options.captureIp !== true) {
       remoteString += '_anonymize';
     }
-    _.set(item, 'data.request', {
-      url: window.location.href,
-      query_string: window.location.search,
-      user_ip: remoteString
-    });
+    if (remoteString) requestInfo.user_ip = remoteString;
+
+    if (Object.keys(requestInfo).length > 0) {
+      _.set(item, 'data.request', requestInfo);
+    }
+
     callback(null, item);
   };
 }

package/src/browser/transport/fetch.js

@@ -0,0 +1,35 @@
+var logger = require('../logger');
+var _ = require('../../utility');
+
+function makeFetchRequest(accessToken, url, method, data, callback, timeout) {
+  var controller;
+  var timeoutId;
+
+  if(_.isFiniteNumber(timeout)) {
+    controller = new AbortController();
+    timeoutId = setTimeout(() => controller.abort(), timeout);
+  }
+
+  fetch(url, {
+    method: method,
+    headers: {
+      'Content-Type': 'application/json',
+      'X-Rollbar-Access-Token': accessToken,
+      signal: controller && controller.signal
+    },
+    body: data,
+  })
+  .then((response) => {
+    if (timeoutId) clearTimeout(timeoutId);
+    return response.json();
+  })
+  .then((data) => {
+    callback(null, data);
+  })
+  .catch((error) => {
+    logger.error(error.message);
+    callback(error);
+  });
+}
+
+module.exports = makeFetchRequest;

package/src/browser/transport/xhr.js

@@ -0,0 +1,159 @@
+/*global XDomainRequest*/
+
+var _ = require('../../utility');
+var logger = require('../logger');
+
+function makeXhrRequest(accessToken, url, method, data, callback, requestFactory, timeout) {
+  var request;
+  if (requestFactory) {
+    request = requestFactory();
+  } else {
+    request = _createXMLHTTPObject();
+  }
+  if (!request) {
+    // Give up, no way to send requests
+    return callback(new Error('No way to send a request'));
+  }
+  try {
+    try {
+      var onreadystatechange = function() {
+        try {
+          if (onreadystatechange && request.readyState === 4) {
+            onreadystatechange = undefined;
+
+            var parseResponse = _.jsonParse(request.responseText);
+            if (_isSuccess(request)) {
+              callback(parseResponse.error, parseResponse.value);
+              return;
+            } else if (_isNormalFailure(request)) {
+              if (request.status === 403) {
+                // likely caused by using a server access token
+                var message = parseResponse.value && parseResponse.value.message;
+                logger.error(message);
+              }
+              // return valid http status codes
+              callback(new Error(String(request.status)));
+            } else {
+              // IE will return a status 12000+ on some sort of connection failure,
+              // so we return a blank error
+              // http://msdn.microsoft.com/en-us/library/aa383770%28VS.85%29.aspx
+              var msg = 'XHR response had no status code (likely connection failure)';
+              callback(_newRetriableError(msg));
+            }
+          }
+        } catch (ex) {
+          //jquery source mentions firefox may error out while accessing the
+          //request members if there is a network error
+          //https://github.com/jquery/jquery/blob/a938d7b1282fc0e5c52502c225ae8f0cef219f0a/src/ajax/xhr.js#L111
+          var exc;
+          if (ex && ex.stack) {
+            exc = ex;
+          } else {
+            exc = new Error(ex);
+          }
+          callback(exc);
+        }
+      };
+
+      request.open(method, url, true);
+      if (request.setRequestHeader) {
+        request.setRequestHeader('Content-Type', 'application/json');
+        request.setRequestHeader('X-Rollbar-Access-Token', accessToken);
+      }
+
+      if(_.isFiniteNumber(timeout)) {
+        request.timeout = timeout;
+      }
+
+      request.onreadystatechange = onreadystatechange;
+      request.send(data);
+    } catch (e1) {
+      // Sending using the normal xmlhttprequest object didn't work, try XDomainRequest
+      if (typeof XDomainRequest !== 'undefined') {
+
+        // Assume we are in a really old browser which has a bunch of limitations:
+        // http://blogs.msdn.com/b/ieinternals/archive/2010/05/13/xdomainrequest-restrictions-limitations-and-workarounds.aspx
+
+        // Extreme paranoia: if we have XDomainRequest then we have a window, but just in case
+        if (!window || !window.location) {
+          return callback(new Error('No window available during request, unknown environment'));
+        }
+
+        // If the current page is http, try and send over http
+        if (window.location.href.substring(0, 5) === 'http:' && url.substring(0, 5) === 'https') {
+          url = 'http' + url.substring(5);
+        }
+
+        var xdomainrequest = new XDomainRequest();
+        xdomainrequest.onprogress = function() {};
+        xdomainrequest.ontimeout = function() {
+          var msg = 'Request timed out';
+          var code = 'ETIMEDOUT';
+          callback(_newRetriableError(msg, code));
+        };
+        xdomainrequest.onerror = function() {
+          callback(new Error('Error during request'));
+        };
+        xdomainrequest.onload = function() {
+          var parseResponse = _.jsonParse(xdomainrequest.responseText);
+          callback(parseResponse.error, parseResponse.value);
+        };
+        xdomainrequest.open(method, url, true);
+        xdomainrequest.send(data);
+      } else {
+        callback(new Error('Cannot find a method to transport a request'));
+      }
+    }
+  } catch (e2) {
+    callback(e2);
+  }
+}
+
+function _createXMLHTTPObject() {
+  /* global ActiveXObject:false */
+
+  var factories = [
+    function () {
+      return new XMLHttpRequest();
+    },
+    function () {
+      return new ActiveXObject('Msxml2.XMLHTTP');
+    },
+    function () {
+      return new ActiveXObject('Msxml3.XMLHTTP');
+    },
+    function () {
+      return new ActiveXObject('Microsoft.XMLHTTP');
+    }
+  ];
+  var xmlhttp;
+  var i;
+  var numFactories = factories.length;
+  for (i = 0; i < numFactories; i++) {
+    /* eslint-disable no-empty */
+    try {
+      xmlhttp = factories[i]();
+      break;
+    } catch (e) {
+      // pass
+    }
+    /* eslint-enable no-empty */
+  }
+  return xmlhttp;
+}
+
+function _isSuccess(r) {
+  return r && r.status && r.status === 200;
+}
+
+function _isNormalFailure(r) {
+  return r && _.isType(r.status, 'number') && r.status >= 400 && r.status < 600;
+}
+
+function _newRetriableError(message, code) {
+  var err = new Error(message);
+  err.code = code || 'ENOTFOUND';
+  return err;
+}
+
+module.exports = makeXhrRequest;

package/src/browser/transport.js

@@ -1,7 +1,6 @@
-/*global XDomainRequest*/
-
 var _ = require('../utility');
-var logger = require('./logger');
+var makeFetchRequest = require('./transport/fetch');
+var makeXhrRequest = require('./transport/xhr');
 
 /*
  * accessToken may be embedded in payload but that should not
@@ -13,6 +12,7 @@ var logger = require('./logger');
  *   path
  *   port
  *   method
+ *   transport ('xhr' | 'fetch')
  * }
  *
  *  params is an object containing key/value pairs. These
@@ -32,7 +32,9 @@ Transport.prototype.get = function(accessToken, options, params, callback, reque
 
   var method = 'GET';
   var url = _.formatUrl(options);
-  _makeZoneRequest(accessToken, url, method, null, callback, requestFactory, options.timeout);
+  this._makeZoneRequest(
+    accessToken, url, method, null, callback, requestFactory, options.timeout, options.transport
+  );
 }
 
 Transport.prototype.post = function(accessToken, options, payload, callback, requestFactory) {
@@ -57,7 +59,9 @@ Transport.prototype.post = function(accessToken, options, payload, callback, req
   var writeData = stringifyResult.value;
   var method = 'POST';
   var url = _.formatUrl(options);
-  _makeZoneRequest(accessToken, url, method, writeData, callback, requestFactory, options.timeout);
+  this._makeZoneRequest(
+    accessToken, url, method, writeData, callback, requestFactory, options.timeout, options.transport
+  );
 }
 
 Transport.prototype.postJsonPayload = function (accessToken, options, jsonPayload, callback, requestFactory) {
@@ -67,7 +71,9 @@ Transport.prototype.postJsonPayload = function (accessToken, options, jsonPayloa
 
   var method = 'POST';
   var url = _.formatUrl(options);
-  _makeZoneRequest(accessToken, url, method, jsonPayload, callback, requestFactory, options.timeout);
+  this._makeZoneRequest(
+    accessToken, url, method, jsonPayload, callback, requestFactory, options.timeout, options.transport
+  );
 }
 
 
@@ -75,7 +81,7 @@ Transport.prototype.postJsonPayload = function (accessToken, options, jsonPayloa
 // so Angular change detection isn't triggered on each API call.
 // This is the equivalent of runOutsideAngular().
 //
-function _makeZoneRequest() {
+Transport.prototype._makeZoneRequest = function () {
   var gWindow = ((typeof window != 'undefined') && window) || ((typeof self != 'undefined') && self);
   var currentZone = gWindow && gWindow.Zone && gWindow.Zone.current;
   var args = Array.prototype.slice.call(arguments);
@@ -83,10 +89,24 @@ function _makeZoneRequest() {
   if (currentZone && currentZone._name === 'angular') {
     var rootZone = currentZone._parent;
     rootZone.run(function () {
-      _makeRequest.apply(undefined, args);
+      this._makeRequest.apply(undefined, args);
     });
   } else {
-    _makeRequest.apply(undefined, args);
+    this._makeRequest.apply(undefined, args);
+  }
+}
+
+Transport.prototype._makeRequest = function (
+  accessToken, url, method, data, callback, requestFactory, timeout, transport
+) {
+  if (typeof RollbarProxy !== 'undefined') {
+    return _proxyRequest(data, callback);
+  }
+
+  if (transport === 'fetch') {
+    makeFetchRequest(accessToken, url, method, data, callback, timeout)
+  } else {
+    makeXhrRequest(accessToken, url, method, data, callback, requestFactory, timeout)
   }
 }
 
@@ -102,161 +122,4 @@ function _proxyRequest(json, callback) {
   );
 }
 
-function _makeRequest(accessToken, url, method, data, callback, requestFactory, timeout) {
-  if (typeof RollbarProxy !== 'undefined') {
-    return _proxyRequest(data, callback);
-  }
-
-  var request;
-  if (requestFactory) {
-    request = requestFactory();
-  } else {
-    request = _createXMLHTTPObject();
-  }
-  if (!request) {
-    // Give up, no way to send requests
-    return callback(new Error('No way to send a request'));
-  }
-  try {
-    try {
-      var onreadystatechange = function() {
-        try {
-          if (onreadystatechange && request.readyState === 4) {
-            onreadystatechange = undefined;
-
-            var parseResponse = _.jsonParse(request.responseText);
-            if (_isSuccess(request)) {
-              callback(parseResponse.error, parseResponse.value);
-              return;
-            } else if (_isNormalFailure(request)) {
-              if (request.status === 403) {
-                // likely caused by using a server access token
-                var message = parseResponse.value && parseResponse.value.message;
-                logger.error(message);
-              }
-              // return valid http status codes
-              callback(new Error(String(request.status)));
-            } else {
-              // IE will return a status 12000+ on some sort of connection failure,
-              // so we return a blank error
-              // http://msdn.microsoft.com/en-us/library/aa383770%28VS.85%29.aspx
-              var msg = 'XHR response had no status code (likely connection failure)';
-              callback(_newRetriableError(msg));
-            }
-          }
-        } catch (ex) {
-          //jquery source mentions firefox may error out while accessing the
-          //request members if there is a network error
-          //https://github.com/jquery/jquery/blob/a938d7b1282fc0e5c52502c225ae8f0cef219f0a/src/ajax/xhr.js#L111
-          var exc;
-          if (ex && ex.stack) {
-            exc = ex;
-          } else {
-            exc = new Error(ex);
-          }
-          callback(exc);
-        }
-      };
-
-      request.open(method, url, true);
-      if (request.setRequestHeader) {
-        request.setRequestHeader('Content-Type', 'application/json');
-        request.setRequestHeader('X-Rollbar-Access-Token', accessToken);
-      }
-
-      if(_.isFiniteNumber(timeout)) {
-        request.timeout = timeout;
-      }
-
-      request.onreadystatechange = onreadystatechange;
-      request.send(data);
-    } catch (e1) {
-      // Sending using the normal xmlhttprequest object didn't work, try XDomainRequest
-      if (typeof XDomainRequest !== 'undefined') {
-
-        // Assume we are in a really old browser which has a bunch of limitations:
-        // http://blogs.msdn.com/b/ieinternals/archive/2010/05/13/xdomainrequest-restrictions-limitations-and-workarounds.aspx
-
-        // Extreme paranoia: if we have XDomainRequest then we have a window, but just in case
-        if (!window || !window.location) {
-          return callback(new Error('No window available during request, unknown environment'));
-        }
-
-        // If the current page is http, try and send over http
-        if (window.location.href.substring(0, 5) === 'http:' && url.substring(0, 5) === 'https') {
-          url = 'http' + url.substring(5);
-        }
-
-        var xdomainrequest = new XDomainRequest();
-        xdomainrequest.onprogress = function() {};
-        xdomainrequest.ontimeout = function() {
-          var msg = 'Request timed out';
-          var code = 'ETIMEDOUT';
-          callback(_newRetriableError(msg, code));
-        };
-        xdomainrequest.onerror = function() {
-          callback(new Error('Error during request'));
-        };
-        xdomainrequest.onload = function() {
-          var parseResponse = _.jsonParse(xdomainrequest.responseText);
-          callback(parseResponse.error, parseResponse.value);
-        };
-        xdomainrequest.open(method, url, true);
-        xdomainrequest.send(data);
-      } else {
-        callback(new Error('Cannot find a method to transport a request'));
-      }
-    }
-  } catch (e2) {
-    callback(e2);
-  }
-}
-
-function _createXMLHTTPObject() {
-  /* global ActiveXObject:false */
-
-  var factories = [
-    function () {
-      return new XMLHttpRequest();
-    },
-    function () {
-      return new ActiveXObject('Msxml2.XMLHTTP');
-    },
-    function () {
-      return new ActiveXObject('Msxml3.XMLHTTP');
-    },
-    function () {
-      return new ActiveXObject('Microsoft.XMLHTTP');
-    }
-  ];
-  var xmlhttp;
-  var i;
-  var numFactories = factories.length;
-  for (i = 0; i < numFactories; i++) {
-    /* eslint-disable no-empty */
-    try {
-      xmlhttp = factories[i]();
-      break;
-    } catch (e) {
-      // pass
-    }
-    /* eslint-enable no-empty */
-  }
-  return xmlhttp;
-}
-
-function _isSuccess(r) {
-  return r && r.status && r.status === 200;
-}
-
-function _isNormalFailure(r) {
-  return r && _.isType(r.status, 'number') && r.status >= 400 && r.status < 600;
-}
-
-function _newRetriableError(message, code) {
-  var err = new Error(message);
-  err.code = code || 'ENOTFOUND';
-  return err;
-}
-
 module.exports = Transport;

package/src/defaults.js

@@ -1,5 +1,5 @@
 module.exports = {
-  version: '2.25.1',
+  version: '2.26.0',
   endpoint: 'api.rollbar.com/api/1/item/',
   logLevel: 'debug',
   reportLevel: 'debug',

package/src/utility/headers.js

@@ -0,0 +1,94 @@
+/*
+ * headers - Detect when fetch Headers are undefined and use a partial polyfill.
+ *
+ * A full polyfill is not used in order to keep package size as small as possible.
+ * Since this is only used internally and is not added to the window object,
+ * the full interface doesn't need to be supported.
+ *
+ * This implementation is modified from whatwg-fetch:
+ * https://github.com/github/fetch
+ */
+function headers(headers) {
+  if (typeof Headers === 'undefined') {
+    return new FetchHeaders(headers);
+  }
+
+  return new Headers(headers);
+}
+
+function normalizeName(name) {
+  if (typeof name !== 'string') {
+    name = String(name)
+  }
+  return name.toLowerCase()
+}
+
+function normalizeValue(value) {
+  if (typeof value !== 'string') {
+    value = String(value)
+  }
+  return value
+}
+
+function iteratorFor(items) {
+  var iterator = {
+    next: function() {
+      var value = items.shift()
+      return {done: value === undefined, value: value}
+    }
+  }
+
+  return iterator
+}
+
+function FetchHeaders(headers) {
+  this.map = {}
+
+  if (headers instanceof FetchHeaders) {
+    headers.forEach(function(value, name) {
+      this.append(name, value)
+    }, this)
+  } else if (Array.isArray(headers)) {
+    headers.forEach(function(header) {
+      this.append(header[0], header[1])
+    }, this)
+  } else if (headers) {
+    Object.getOwnPropertyNames(headers).forEach(function(name) {
+      this.append(name, headers[name])
+    }, this)
+  }
+}
+
+FetchHeaders.prototype.append = function(name, value) {
+  name = normalizeName(name)
+  value = normalizeValue(value)
+  var oldValue = this.map[name]
+  this.map[name] = oldValue ? oldValue + ', ' + value : value
+}
+
+FetchHeaders.prototype.get = function(name) {
+  name = normalizeName(name)
+  return this.has(name) ? this.map[name] : null
+}
+
+FetchHeaders.prototype.has = function(name) {
+  return this.map.hasOwnProperty(normalizeName(name))
+}
+
+FetchHeaders.prototype.forEach = function(callback, thisArg) {
+  for (var name in this.map) {
+    if (this.map.hasOwnProperty(name)) {
+      callback.call(thisArg, this.map[name], name, this)
+    }
+  }
+}
+
+FetchHeaders.prototype.entries = function() {
+  var items = []
+  this.forEach(function(value, name) {
+    items.push([name, value])
+  })
+  return iteratorFor(items)
+}
+
+module.exports = headers;