3.12.2

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 3.12.2
+* BUGFIX: Fix Raven.js not capturing IE8 errors. See: https://github.com/getsentry/raven-js/pull/883
+
 ## 3.12.1
 * BUGFIX: Fix Raven.js not properly catching some thrown messages, objects. See: https://github.com/getsentry/raven-js/pull/872
 

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "raven-js",
-  "version": "3.12.1",
+  "version": "3.12.2",
   "dependencies": {},
   "main": "dist/raven.js",
   "ignore": [

dist/plugins/angular.js

@@ -1,4 +1,4 @@
-/*! Raven.js 3.12.1 (3600a05) | github.com/getsentry/raven-js */
+/*! Raven.js 3.12.2 (d911cb7) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit

dist/plugins/console.js

@@ -1,4 +1,4 @@
-/*! Raven.js 3.12.1 (3600a05) | github.com/getsentry/raven-js */
+/*! Raven.js 3.12.2 (d911cb7) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit

dist/plugins/ember.js

@@ -1,4 +1,4 @@
-/*! Raven.js 3.12.1 (3600a05) | github.com/getsentry/raven-js */
+/*! Raven.js 3.12.2 (d911cb7) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit

dist/plugins/require.js

@@ -1,4 +1,4 @@
-/*! Raven.js 3.12.1 (3600a05) | github.com/getsentry/raven-js */
+/*! Raven.js 3.12.2 (d911cb7) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit

dist/plugins/vue.js

@@ -1,4 +1,4 @@
-/*! Raven.js 3.12.1 (3600a05) | github.com/getsentry/raven-js */
+/*! Raven.js 3.12.2 (d911cb7) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit

dist/raven.js

@@ -1,4 +1,4 @@
-/*! Raven.js 3.12.1 (3600a05) | github.com/getsentry/raven-js */
+/*! Raven.js 3.12.2 (d911cb7) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit
@@ -11,35 +11,6 @@
  */
 
 (function(f){if(typeof exports==="object"&&typeof module!=="undefined"){module.exports=f()}else if(typeof define==="function"&&define.amd){define([],f)}else{var g;if(typeof window!=="undefined"){g=window}else if(typeof global!=="undefined"){g=global}else if(typeof self!=="undefined"){g=self}else{g=this}g.Raven = f()}})(function(){var define,module,exports;return (function e(t,n,r){function s(o,u){if(!n[o]){if(!t[o]){var a=typeof require=="function"&&require;if(!u&&a)return a(o,!0);if(i)return i(o,!0);var f=new Error("Cannot find module '"+o+"'");throw f.code="MODULE_NOT_FOUND",f}var l=n[o]={exports:{}};t[o][0].call(l.exports,function(e){var n=t[o][1][e];return s(n?n:e)},l,l.exports,e,t,n,r)}return n[o].exports}var i=typeof require=="function"&&require;for(var o=0;o<r.length;o++)s(r[o]);return s})({1:[function(_dereq_,module,exports){
-exports = module.exports = stringify
-exports.getSerialize = serializer
-
-function stringify(obj, replacer, spaces, cycleReplacer) {
-  return JSON.stringify(obj, serializer(replacer, cycleReplacer), spaces)
-}
-
-function serializer(replacer, cycleReplacer) {
-  var stack = [], keys = []
-
-  if (cycleReplacer == null) cycleReplacer = function(key, value) {
-    if (stack[0] === value) return "[Circular ~]"
-    return "[Circular ~." + keys.slice(0, stack.indexOf(value)).join(".") + "]"
-  }
-
-  return function(key, value) {
-    if (stack.length > 0) {
-      var thisPos = stack.indexOf(this)
-      ~thisPos ? stack.splice(thisPos + 1) : stack.push(this)
-      ~thisPos ? keys.splice(thisPos, Infinity, key) : keys.push(key)
-      if (~stack.indexOf(value)) value = cycleReplacer.call(this, key, value)
-    }
-    else stack.push(value)
-
-    return replacer == null ? value : replacer.call(this, key, value)
-  }
-}
-
-},{}],2:[function(_dereq_,module,exports){
 'use strict';
 
 function RavenConfigError(message) {
@@ -51,7 +22,7 @@ RavenConfigError.prototype.constructor = RavenConfigError;
 
 module.exports = RavenConfigError;
 
-},{}],3:[function(_dereq_,module,exports){
+},{}],2:[function(_dereq_,module,exports){
 'use strict';
 
 var wrapMethod = function(console, level, callback) {
@@ -90,21 +61,20 @@ module.exports = {
     wrapMethod: wrapMethod
 };
 
-},{}],4:[function(_dereq_,module,exports){
+},{}],3:[function(_dereq_,module,exports){
 (function (global){
 /*global XDomainRequest:false, __DEV__:false*/
 'use strict';
 
-var TraceKit = _dereq_(7);
-var RavenConfigError = _dereq_(2);
-var utils = _dereq_(6);
+var TraceKit = _dereq_(6);
+var stringify = _dereq_(7);
+var RavenConfigError = _dereq_(1);
+var utils = _dereq_(5);
 
 var isError = utils.isError,
     isObject = utils.isObject;
 
-var stringify = _dereq_(1);
-
-var wrapConsoleMethod = _dereq_(3).wrapMethod;
+var wrapConsoleMethod = _dereq_(2).wrapMethod;
 
 var dsnKeys = 'source protocol user pass host port path'.split(' '),
     dsnPattern = /^(?:(\w+):)?\/\/(?:(\w+)(:\w+)?@)?([\w\.-]+)(?::(\d+))?(\/.*)/;
@@ -181,7 +151,7 @@ Raven.prototype = {
     // webpack (using a build step causes webpack #1617). Grunt verifies that
     // this value matches package.json during build.
     //   See: https://github.com/getsentry/raven-js/issues/465
-    VERSION: '3.12.1',
+    VERSION: '3.12.2',
 
     debug: false,
 
@@ -1679,24 +1649,18 @@ Raven.prototype = {
         if (!hasCORS) return;
 
         var url = opts.url;
-        function handler() {
-            if (request.status === 200) {
-                if (opts.onSuccess) {
-                    opts.onSuccess();
-                }
-            } else if (opts.onError) {
-                var err = new Error('Sentry error code: ' + request.status);
-                err.request = request;
-                opts.onError(err);
-            }
-        }
 
         if ('withCredentials' in request) {
             request.onreadystatechange = function () {
                 if (request.readyState !== 4) {
                     return;
+                } else if (request.status === 200) {
+                    opts.onSuccess && opts.onSuccess();
+                } else if (opts.onError) {
+                    var err = new Error('Sentry error code: ' + request.status);
+                    err.request = request;
+                    opts.onError(err);
                 }
-                handler();
             };
         } else {
             request = new XDomainRequest();
@@ -1705,7 +1669,16 @@ Raven.prototype = {
             url = url.replace(/^https?:/, '');
 
             // onreadystatechange not supported by XDomainRequest
-            request.onload = handler;
+            if (opts.onSuccess) {
+                request.onload = opts.onSuccess;
+            }
+            if (opts.onError) {
+                request.onerror = function () {
+                    var err = new Error('Sentry error code: XDomainRequest');
+                    err.request = request;
+                    opts.onError(err);
+                }
+            }
         }
 
         // NOTE: auth is intentionally sent as part of query string (NOT as custom
@@ -2059,7 +2032,7 @@ Raven.prototype.setReleaseContext = Raven.prototype.setRelease;
 module.exports = Raven;
 
 }).call(this,typeof global !== "undefined" ? global : typeof self !== "undefined" ? self : typeof window !== "undefined" ? window : {})
-},{"1":1,"2":2,"3":3,"6":6,"7":7}],5:[function(_dereq_,module,exports){
+},{"1":1,"2":2,"5":5,"6":6,"7":7}],4:[function(_dereq_,module,exports){
 (function (global){
 /**
  * Enforces a single instance of the Raven client, and the
@@ -2069,7 +2042,7 @@ module.exports = Raven;
 
 'use strict';
 
-var RavenConstructor = _dereq_(4);
+var RavenConstructor = _dereq_(3);
 
 // This is to be defensive in environments where window does not exist (see https://github.com/getsentry/raven-js/pull/785)
 var _window = typeof window !== 'undefined' ? window
@@ -2096,7 +2069,7 @@ Raven.afterLoad();
 module.exports = Raven;
 
 }).call(this,typeof global !== "undefined" ? global : typeof self !== "undefined" ? self : typeof window !== "undefined" ? window : {})
-},{"4":4}],6:[function(_dereq_,module,exports){
+},{"3":3}],5:[function(_dereq_,module,exports){
 'use strict';
 
 function isObject(what) {
@@ -2117,11 +2090,11 @@ module.exports = {
     isObject: isObject,
     isError: isError
 };
-},{}],7:[function(_dereq_,module,exports){
+},{}],6:[function(_dereq_,module,exports){
 (function (global){
 'use strict';
 
-var utils = _dereq_(6);
+var utils = _dereq_(5);
 
 /*
  TraceKit - Cross brower stack traces
@@ -2737,5 +2710,54 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
 module.exports = TraceKit;
 
 }).call(this,typeof global !== "undefined" ? global : typeof self !== "undefined" ? self : typeof window !== "undefined" ? window : {})
-},{"6":6}]},{},[5])(5)
+},{"5":5}],7:[function(_dereq_,module,exports){
+'use strict';
+
+/*
+ json-stringify-safe
+ Like JSON.stringify, but doesn't throw on circular references.
+
+ Originally forked from https://github.com/isaacs/json-stringify-safe
+ version 5.0.1 on 3/8/2017 and modified for IE8 compatibility.
+ Tests for this are in test/vendor.
+
+ ISC license: https://github.com/isaacs/json-stringify-safe/blob/master/LICENSE
+*/
+
+exports = module.exports = stringify
+exports.getSerialize = serializer
+
+function indexOf(haystack, needle) {
+  for (var i = 0; i < haystack.length; ++i) {
+    if (haystack[i] === needle) return i;
+  }
+  return -1;
+}
+
+function stringify(obj, replacer, spaces, cycleReplacer) {
+  return JSON.stringify(obj, serializer(replacer, cycleReplacer), spaces)
+}
+
+function serializer(replacer, cycleReplacer) {
+  var stack = [], keys = []
+
+  if (cycleReplacer == null) cycleReplacer = function(key, value) {
+    if (stack[0] === value) return '[Circular ~]'
+    return '[Circular ~.' + keys.slice(0, indexOf(stack, value)).join('.') + ']'
+  }
+
+  return function(key, value) {
+    if (stack.length > 0) {
+      var thisPos = indexOf(stack, this);
+      ~thisPos ? stack.splice(thisPos + 1) : stack.push(this)
+      ~thisPos ? keys.splice(thisPos, Infinity, key) : keys.push(key)
+      if (~indexOf(stack, value)) value = cycleReplacer.call(this, key, value)
+    }
+    else stack.push(value)
+
+    return replacer == null ? value : replacer.call(this, key, value)
+  }
+}
+
+},{}]},{},[4])(4)
 });

dist/sri.json

@@ -1,20 +1,20 @@
 {
   "@dist/raven.js": {
     "hashes": {
-      "sha256": "lo47JDw5iui9P+8BVkoYgcXash9DqsapFwWuBx47xg4=",
-      "sha512": "4VSTdDWObyBxQUbuBYSOd2MZlXM9F1CsRsC1LTyArrGbWiqHVIEEv8vuprjLF+5t3rGI9qJf4mMeBFyircjiIw=="
+      "sha256": "0jz3OL8CfgEWZEXV15V03fJnv6QsV5Oh8U98Ab3ZB6s=",
+      "sha512": "rD0+yqgIv0MsBrZNAilL7uzutAObf20GpYeGdsm6yJ1e6UyfZmwBuBA9SWXfta228prhZ1kbPqqjFDtIMglH7Q=="
     },
     "type": null,
-    "integrity": "sha256-lo47JDw5iui9P+8BVkoYgcXash9DqsapFwWuBx47xg4= sha512-4VSTdDWObyBxQUbuBYSOd2MZlXM9F1CsRsC1LTyArrGbWiqHVIEEv8vuprjLF+5t3rGI9qJf4mMeBFyircjiIw==",
+    "integrity": "sha256-0jz3OL8CfgEWZEXV15V03fJnv6QsV5Oh8U98Ab3ZB6s= sha512-rD0+yqgIv0MsBrZNAilL7uzutAObf20GpYeGdsm6yJ1e6UyfZmwBuBA9SWXfta228prhZ1kbPqqjFDtIMglH7Q==",
     "path": "dist/raven.js"
   },
   "@dist/raven.min.js": {
     "hashes": {
-      "sha256": "8uqNKvEWACUnXXdaGWrX2mEz3TEE2h+m5dcC8DJVyCI=",
-      "sha512": "LobR1Ghrmpm2vODObzjVZuWLajD0wEl58MWvfBzfeSQA4K0VrZSNBpwlEOpKhNPD/qwdSPAoKdXWilbAzryNNA=="
+      "sha256": "6bH9yJOSxvm/1apXVXlIm9ps7+zKz6R4VoUADoCiEiI=",
+      "sha512": "4iLoJOdPpE4eHuC1+8xhUvYbJs7R38qNKBaLR39dg/I1kQN3XepMXw1vjwMBTMnkqKPY3UcMRgIw5t9IHcYo9g=="
     },
     "type": null,
-    "integrity": "sha256-8uqNKvEWACUnXXdaGWrX2mEz3TEE2h+m5dcC8DJVyCI= sha512-LobR1Ghrmpm2vODObzjVZuWLajD0wEl58MWvfBzfeSQA4K0VrZSNBpwlEOpKhNPD/qwdSPAoKdXWilbAzryNNA==",
+    "integrity": "sha256-6bH9yJOSxvm/1apXVXlIm9ps7+zKz6R4VoUADoCiEiI= sha512-4iLoJOdPpE4eHuC1+8xhUvYbJs7R38qNKBaLR39dg/I1kQN3XepMXw1vjwMBTMnkqKPY3UcMRgIw5t9IHcYo9g==",
     "path": "dist/raven.min.js"
   }
 }

docs/sentry-doc-config.json

@@ -66,6 +66,6 @@
     }
   },
   "vars": {
-    "RAVEN_VERSION": "3.12.1"
+    "RAVEN_VERSION": "3.12.2"
   }
 }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "raven-js",
-  "version": "3.12.1",
+  "version": "3.12.2",
   "license": "BSD-2-Clause",
   "homepage": "https://github.com/getsentry/raven-js",
   "scripts": {

src/raven.js

@@ -86,7 +86,7 @@ Raven.prototype = {
     // webpack (using a build step causes webpack #1617). Grunt verifies that
     // this value matches package.json during build.
     //   See: https://github.com/getsentry/raven-js/issues/465
-    VERSION: '3.12.1',
+    VERSION: '3.12.2',
 
     debug: false,
 

test/raven.test.js

@@ -1010,7 +1010,7 @@ describe('globals', function() {
                 extra: {'session:duration': 100},
             });
             assert.deepEqual(opts.auth, {
-                sentry_client: 'raven-js/3.12.1',
+                sentry_client: 'raven-js/3.12.2',
                 sentry_key: 'abc',
                 sentry_version: '7'
             });
@@ -1057,7 +1057,7 @@ describe('globals', function() {
                 extra: {'session:duration': 100},
             });
             assert.deepEqual(opts.auth, {
-                sentry_client: 'raven-js/3.12.1',
+                sentry_client: 'raven-js/3.12.2',
                 sentry_key: 'abc',
                 sentry_secret: 'def',
                 sentry_version: '7'