3.1.1 (#624)

benvinegar · web-flow · commit aa5a6f395b7e · 2016-06-23T13:56:21.000-07:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+## 3.1.1
+* BUGFIX: Fix message truncation occurring before dataCallback is invoked. See: https://github.com/getsentry/raven-js/issues/605
+* BUGFIX: Fix pushState error in Chrome Apps. See: https://github.com/getsentry/raven-js/issues/601
+* BUGFIX: Fix error in addEventListener call affecting very old Firefox versions. See: https://github.com/getsentry/raven-js/issues/603
+
 ## 3.1.0
 * NEW: Added TypeScript declaration file for compatibility with TypeScript projects. See: https://github.com/getsentry/raven-js/pull/610
 
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "raven-js",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "dependencies": {},
   "main": "dist/raven.js",
   "ignore": [
diff --git a/dist/plugins/angular.js b/dist/plugins/angular.js
@@ -1,4 +1,4 @@
-/*! Raven.js 3.1.0 (d781478) | github.com/getsentry/raven-js */
+/*! Raven.js 3.1.1 (34f456e) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit
diff --git a/dist/plugins/angular.min.js b/dist/plugins/angular.min.js
diff --git a/dist/plugins/console.js b/dist/plugins/console.js
@@ -1,4 +1,4 @@
-/*! Raven.js 3.1.0 (d781478) | github.com/getsentry/raven-js */
+/*! Raven.js 3.1.1 (34f456e) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit
diff --git a/dist/plugins/console.min.js b/dist/plugins/console.min.js
diff --git a/dist/plugins/ember.js b/dist/plugins/ember.js
@@ -1,4 +1,4 @@
-/*! Raven.js 3.1.0 (d781478) | github.com/getsentry/raven-js */
+/*! Raven.js 3.1.1 (34f456e) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit
diff --git a/dist/plugins/ember.min.js b/dist/plugins/ember.min.js
diff --git a/dist/plugins/require.js b/dist/plugins/require.js
@@ -1,4 +1,4 @@
-/*! Raven.js 3.1.0 (d781478) | github.com/getsentry/raven-js */
+/*! Raven.js 3.1.1 (34f456e) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit
diff --git a/dist/plugins/require.min.js b/dist/plugins/require.min.js
diff --git a/dist/raven.js b/dist/raven.js
@@ -1,4 +1,4 @@
-/*! Raven.js 3.1.0 (d781478) | github.com/getsentry/raven-js */
+/*! Raven.js 3.1.1 (34f456e) | github.com/getsentry/raven-js */
 
 /*
  * Includes TraceKit
@@ -150,7 +150,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.1.0',
+    VERSION: '3.1.1',
 
     debug: false,
 
@@ -878,8 +878,8 @@ Raven.prototype = {
         // to the document. Do this before we instrument addEventListener.
         if (this._hasDocument) {
             if (document.addEventListener) {
-                document.addEventListener('click', self._breadcrumbEventHandler('click'));
-                document.addEventListener('keypress', self._keypressEventHandler());
+                document.addEventListener('click', self._breadcrumbEventHandler('click'), false);
+                document.addEventListener('keypress', self._keypressEventHandler(), false);
             }
             else {
                 // IE8 Compatibility
@@ -953,7 +953,13 @@ Raven.prototype = {
         }
 
         // record navigation (URL) changes
-        if ('history' in window && history.pushState) {
+        // NOTE: in Chrome App environment, touching history.pushState, *even inside
+        //       a try/catch block*, will cause Chrome to output an error to console.error
+        // borrowed from: https://github.com/angular/angular.js/pull/13945/files
+        var chrome = window.chrome;
+        var isChromePackagedApp = chrome && chrome.app && chrome.app.runtime;
+        var hasPushState = !isChromePackagedApp && window.history && history.pushState;
+        if (hasPushState) {
             // TODO: remove onpopstate handler on uninstall()
             var oldOnPopState = window.onpopstate;
             window.onpopstate = function () {
@@ -1125,10 +1131,7 @@ Raven.prototype = {
         if (!!this._globalOptions.ignoreErrors.test && this._globalOptions.ignoreErrors.test(message)) return;
 
         message += '';
-        message = truncate(message, this._globalOptions.maxMessageLength);
-
         fullMessage = (type ? type + ': ' : '') + message;
-        fullMessage = truncate(fullMessage, this._globalOptions.maxMessageLength);
 
         if (frames && frames.length) {
             fileurl = frames[0].filename || fileurl;
diff --git a/dist/raven.min.js b/dist/raven.min.js
diff --git a/dist/raven.min.js.map b/dist/raven.min.js.map
diff --git a/dist/sri.json b/dist/sri.json
@@ -1,20 +1,20 @@
 {
   "@dist/raven.js": {
     "hashes": {
-      "sha256": "9exQEvaM4NkjZiQscZ7vaZZNu0np8tnC9fGGReyxFXg=",
-      "sha512": "t7keBoCkzXX0uqzWc/tkHTMrDZBoOAs1K1decopIZpUsqXDHDdM/Xsho6rFmryJofQ4cVNAV4+nVmuaEJh4juA=="
+      "sha256": "YfJL5HnkAUb6LaT41WdeubM9tHcj1KLNZzv091O0kAA=",
+      "sha512": "rc2JMbKhV8x5vp/NPhCGMsrepxYezWg/Xyvb0saXLKNy8vC5unYVfMXxe6KS9WVhCPOIt12OoZDKDmohxpT60Q=="
     },
     "type": null,
-    "integrity": "sha256-9exQEvaM4NkjZiQscZ7vaZZNu0np8tnC9fGGReyxFXg= sha512-t7keBoCkzXX0uqzWc/tkHTMrDZBoOAs1K1decopIZpUsqXDHDdM/Xsho6rFmryJofQ4cVNAV4+nVmuaEJh4juA==",
+    "integrity": "sha256-YfJL5HnkAUb6LaT41WdeubM9tHcj1KLNZzv091O0kAA= sha512-rc2JMbKhV8x5vp/NPhCGMsrepxYezWg/Xyvb0saXLKNy8vC5unYVfMXxe6KS9WVhCPOIt12OoZDKDmohxpT60Q==",
     "path": "dist/raven.js"
   },
   "@dist/raven.min.js": {
     "hashes": {
-      "sha256": "eOzZlXH2N6ZXIB2rRnQSFPsUINO1W0UobAllDeMyTGI=",
-      "sha512": "5Vq83HbmtmmeJjzwV9m50F1C3Q6VHouqvFJsYxkFytEspXYqoWNIysZ+l9+Xm0lNUBgXvWzzUz9Jj2DTMBO0hw=="
+      "sha256": "kgc2mY6SdVCVixqDm9Jx5zGX+JAIYPWpXY6QJAzLmhE=",
+      "sha512": "0Idcd2pOWVAvkBPiODuHMMq8YxGsrAc2SsFvQEdpQ2QGrPf/9OE2dUlspBBGC5UvD4Y/rbpY74SCVIfQzBBv8g=="
     },
     "type": null,
-    "integrity": "sha256-eOzZlXH2N6ZXIB2rRnQSFPsUINO1W0UobAllDeMyTGI= sha512-5Vq83HbmtmmeJjzwV9m50F1C3Q6VHouqvFJsYxkFytEspXYqoWNIysZ+l9+Xm0lNUBgXvWzzUz9Jj2DTMBO0hw==",
+    "integrity": "sha256-kgc2mY6SdVCVixqDm9Jx5zGX+JAIYPWpXY6QJAzLmhE= sha512-0Idcd2pOWVAvkBPiODuHMMq8YxGsrAc2SsFvQEdpQ2QGrPf/9OE2dUlspBBGC5UvD4Y/rbpY74SCVIfQzBBv8g==",
     "path": "dist/raven.min.js"
   }
 }
diff --git a/docs/config.rst b/docs/config.rst
@@ -237,7 +237,7 @@ Putting it all together
     <body>
         ...
         <script src="jquery.min.js"></script>
-        <script src="https://cdn.ravenjs.com/3.1.0/raven.min.js"></script>
+        <script src="https://cdn.ravenjs.com/3.1.1/raven.min.js"></script>
         <script>
             Raven.config('___PUBLIC_DSN___', {
                 logger: 'my-logger',
diff --git a/docs/index.rst b/docs/index.rst
@@ -26,7 +26,7 @@ scripts.  For all details see :doc:`install`.
 
 .. sourcecode:: html
 
-    <script src="https://cdn.ravenjs.com/3.1.0/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/raven.min.js"></script>
 
 
 Configuring the Client
diff --git a/docs/install.rst b/docs/install.rst
@@ -9,7 +9,7 @@ So for example:
 .. sourcecode:: html
 
     <script src="jquery.js"></script>
-    <script src="https://cdn.ravenjs.com/3.1.0/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/raven.min.js"></script>
     <script>Raven.config('___PUBLIC_DSN___').install();</script>
     <script src="app.js"></script>
 
@@ -28,7 +28,7 @@ Our CDN distributes builds with and without :doc:`integrations <integrations/ind
 
 .. sourcecode:: html
 
-    <script src="https://cdn.ravenjs.com/3.1.0/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/raven.min.js"></script>
 
 This version does not include any plugins. See `ravenjs.com
 <http://ravenjs.com/>`_ for more information about plugins and getting
diff --git a/docs/integrations/angular.rst b/docs/integrations/angular.rst
@@ -24,7 +24,7 @@ Example:
 .. sourcecode:: html
 
     <script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.5/angular.min.js"></script>
-    <script src="https://cdn.ravenjs.com/3.1.0/angular/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/angular/raven.min.js"></script>
     <script>Raven.config('___PUBLIC_DSN___').install();</script>
 
 Note that this CDN build auto-initializes the Angular plugin.
diff --git a/docs/integrations/backbone.rst b/docs/integrations/backbone.rst
@@ -9,7 +9,7 @@ after you load all other external libraries (like jQuery), but before your code.
 
 .. sourcecode:: html
 
-    <script src="https://cdn.ravenjs.com/3.1.0/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/raven.min.js"></script>
 
 Configuring the Client
 ----------------------
diff --git a/docs/integrations/ember.rst b/docs/integrations/ember.rst
@@ -23,7 +23,7 @@ Example:
 .. sourcecode:: html
 
     <script src="http://builds.emberjs.com/tags/v2.3.1/ember.prod.js"></script>
-    <script src="https://cdn.ravenjs.com/3.1.0/ember/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/ember/raven.min.js"></script>
     <script>Raven.config('___PUBLIC_DSN___').install();</script>
 
 Note that this CDN build auto-initializes the Ember plugin.
diff --git a/docs/integrations/react.rst b/docs/integrations/react.rst
@@ -9,7 +9,7 @@ after you load all other external libraries (like jQuery), but before your code.
 
 .. sourcecode:: html
 
-    <script src="https://cdn.ravenjs.com/3.1.0/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.1.1/raven.min.js"></script>
 
 Configuring the Client
 ----------------------
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "raven-js",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "license": "BSD-2-Clause",
   "homepage": "https://getsentry.com",
   "scripts": {
diff --git a/src/raven.js b/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.1.0',
+    VERSION: '3.1.1',
 
     debug: false,
 
diff --git a/test/raven.test.js b/test/raven.test.js
@@ -965,7 +965,7 @@ describe('globals', function() {
                 extra: {'session:duration': 100},
             });
             assert.deepEqual(opts.auth, {
-                sentry_client: 'raven-js/3.1.0',
+                sentry_client: 'raven-js/3.1.1',
                 sentry_key: 'abc',
                 sentry_version: '7'
             });
@@ -1012,7 +1012,7 @@ describe('globals', function() {
                 extra: {'session:duration': 100},
             });
             assert.deepEqual(opts.auth, {
-                sentry_client: 'raven-js/3.1.0',
+                sentry_client: 'raven-js/3.1.1',
                 sentry_key: 'abc',
                 sentry_secret: 'def',
                 sentry_version: '7'

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "raven-js",`
`3`		`- "version": "3.1.0",`
	`3`	`+ "version": "3.1.1",`
`4`	`4`	`"dependencies": {},`
`5`	`5`	`"main": "dist/raven.js",`
`6`	`6`	`"ignore": [`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-/! Raven.js 3.1.0 (d781478) \| github.com/getsentry/raven-js /`
	`1`	`+/! Raven.js 3.1.1 (34f456e) \| github.com/getsentry/raven-js /`
`2`	`2`
`3`	`3`	`/*`
`4`	`4`	`* Includes TraceKit`