Merge pull request #571 from getsentry/3.0.3

benvinegar · benvinegar · commit 1a523953826c · 2016-05-06T09:58:10.000-07:00
3.0.3
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,8 @@
 # Changelog
 
+## 3.0.3
+* BUGFIX: Fix pushState instrumentation breaking on non-string URL args. See: https://github.com/getsentry/raven-js/issues/569  
+
 ## 3.0.2
 * BUGFIX: Fix XMLHttpRequest.prototype.open breaking on non-string `url` arguments. See: https://github.com/getsentry/raven-js/issues/567
 
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "raven-js",
-  "version": "3.0.2",
+  "version": "3.0.3",
   "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.0.2 (da9227a) | github.com/getsentry/raven-js */
+/*! Raven.js 3.0.3 (3738576) | 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.0.2 (da9227a) | github.com/getsentry/raven-js */
+/*! Raven.js 3.0.3 (3738576) | 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.0.2 (da9227a) | github.com/getsentry/raven-js */
+/*! Raven.js 3.0.3 (3738576) | 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.0.2 (da9227a) | github.com/getsentry/raven-js */
+/*! Raven.js 3.0.3 (3738576) | 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.0.2 (da9227a) | github.com/getsentry/raven-js */
+/*! Raven.js 3.0.3 (3738576) | 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.0.2',
+    VERSION: '3.0.3',
 
     debug: false,
 
@@ -974,7 +974,8 @@ Raven.prototype = {
 
                     // url argument is optional
                     if (url) {
-                        self._captureUrlChange(self._lastHref, url);
+                        // coerce to string (this is what pushState does)
+                        self._captureUrlChange(self._lastHref, url + '');
                     }
 
                     return origPushState.apply(this, arguments);
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": "yh0l1izZy8ACTse9vyqoU0sO5pLWH5IkGjQv6Mo5bLs=",
-      "sha512": "gbJtmy+JJcwFhkgk+pjtGYCplYvgHUZCoV+N2EMwqOsKV4LspU0biazBpkZ2Cx9wiXR50hNfYzmyJNhfjXwbFg=="
+      "sha256": "QjY8F5qPj7FAZ9zHS5G4O8047voJ/Y5cx92SRno24uc=",
+      "sha512": "1QM0Ou9Yp+QZ86ZZgqBFLgXFlwi3Igi/Yte1VvgsM9nQnapbyyhd5NdkMBrt73lFCcMXcMSQwEJSRKf4VKRd8A=="
     },
     "type": null,
-    "integrity": "sha256-yh0l1izZy8ACTse9vyqoU0sO5pLWH5IkGjQv6Mo5bLs= sha512-gbJtmy+JJcwFhkgk+pjtGYCplYvgHUZCoV+N2EMwqOsKV4LspU0biazBpkZ2Cx9wiXR50hNfYzmyJNhfjXwbFg==",
+    "integrity": "sha256-QjY8F5qPj7FAZ9zHS5G4O8047voJ/Y5cx92SRno24uc= sha512-1QM0Ou9Yp+QZ86ZZgqBFLgXFlwi3Igi/Yte1VvgsM9nQnapbyyhd5NdkMBrt73lFCcMXcMSQwEJSRKf4VKRd8A==",
     "path": "dist/raven.js"
   },
   "@dist/raven.min.js": {
     "hashes": {
-      "sha256": "Lz811EBXq/vXGUpHazW7Fs3tZpu4Iw8PUm9bMHeiBEI=",
-      "sha512": "TXOh90oJ/4W6hOZGiHb9/1nmk9XTn8pptuynJBgpHY2xiNgoCupsVUjrOVDI6CNZQqeeCMnAjIi9altk2otpzw=="
+      "sha256": "RquNiADCm9QcEuQSH0dRBOBv/KEzrceIvP0bWENzzrU=",
+      "sha512": "Utvtpxczqgj+B0P8w771re39iFYld8tRvWzez6dU8vVjmFwQFb8j3o7tFPgkg4qXw4xgZNLlFLcKReA0U3wDww=="
     },
     "type": null,
-    "integrity": "sha256-Lz811EBXq/vXGUpHazW7Fs3tZpu4Iw8PUm9bMHeiBEI= sha512-TXOh90oJ/4W6hOZGiHb9/1nmk9XTn8pptuynJBgpHY2xiNgoCupsVUjrOVDI6CNZQqeeCMnAjIi9altk2otpzw==",
+    "integrity": "sha256-RquNiADCm9QcEuQSH0dRBOBv/KEzrceIvP0bWENzzrU= sha512-Utvtpxczqgj+B0P8w771re39iFYld8tRvWzez6dU8vVjmFwQFb8j3o7tFPgkg4qXw4xgZNLlFLcKReA0U3wDww==",
     "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.0.2/raven.min.js"></script>
+        <script src="https://cdn.ravenjs.com/3.0.3/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.0.2/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/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.0.2/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/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.0.2/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/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
@@ -27,7 +27,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.0.2/angular/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/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.0.2/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/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.0.2/ember/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/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.0.2/raven.min.js"></script>
+    <script src="https://cdn.ravenjs.com/3.0.3/raven.min.js"></script>
 
 Configuring the Client
 ----------------------
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "raven-js",
-  "version": "3.0.2",
+  "version": "3.0.3",
   "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.0.2',
+    VERSION: '3.0.3',
 
     debug: false,
 
diff --git a/test/raven.test.js b/test/raven.test.js
@@ -1074,7 +1074,7 @@ describe('globals', function() {
                 extra: {'session:duration': 100},
             });
             assert.deepEqual(opts.auth, {
-                sentry_client: 'raven-js/3.0.2',
+                sentry_client: 'raven-js/3.0.3',
                 sentry_key: 'abc',
                 sentry_version: '7'
             });
@@ -1121,7 +1121,7 @@ describe('globals', function() {
                 extra: {'session:duration': 100},
             });
             assert.deepEqual(opts.auth, {
-                sentry_client: 'raven-js/3.0.2',
+                sentry_client: 'raven-js/3.0.3',
                 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.0.2",`
	`3`	`+ "version": "3.0.3",`
`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.0.2 (da9227a) \| github.com/getsentry/raven-js /`
	`1`	`+/! Raven.js 3.0.3 (3738576) \| github.com/getsentry/raven-js /`
`2`	`2`
`3`	`3`	`/*`
`4`	`4`	`* Includes TraceKit`