Merge pull request #14 from talis/ELE-3421-include-url-in-errors

pads · web-flow · commit 2f0d34a2b155 · 2020-10-22T16:31:16.000+01:00
ELE-3421 Include URL in NewRelic error reporting
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## 2.2.1 (2020-10-23)
+
+Improvement:
+
+ - Log the browser URL when logging errors and exceptions
+
 ## 2.1.6 (2019-12-2)
 
 Improvement:
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
     "name": "app-logger-angular",
-    "version": "2.2.0",
+    "version": "2.2.1",
     "main": "./js/logging.js",
     "description": "Client side logging sent to the server",
     "repository": {
diff --git a/js/logging.js b/js/logging.js
@@ -54,7 +54,7 @@ loggingModule.factory(
             }
 
             if (LOGGING_CONFIG.FORWARD_TO_NEWRELIC && $window.NREUM && $window.NREUM.noticeError) {
-                $window.NREUM.noticeError(exception);
+                $window.NREUM.noticeError(exception, { url: $window.location.href });
             }
 
             // check if the config says we should log to the remote, and also if a remote endpoint was specified
@@ -149,7 +149,7 @@ loggingModule.factory(
             }
 
             if (sendToNewRelic && $window.NREUM && $window.NREUM.noticeError) {
-                $window.NREUM.noticeError(message, {desc: desc});
+                $window.NREUM.noticeError(message, { desc: desc, url: $window.location.href });
             }
 
             // check if the config says we should log to the remote, and also if a remote endpoint was specified

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "app-logger-angular",`
`3`		`- "version": "2.2.0",`
	`3`	`+ "version": "2.2.1",`
`4`	`4`	`"main": "./js/logging.js",`
`5`	`5`	`"description": "Client side logging sent to the server",`
`6`	`6`	`"repository": {`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ loggingModule.factory(`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`if (LOGGING_CONFIG.FORWARD_TO_NEWRELIC && $window.NREUM && $window.NREUM.noticeError) {`
`57`		`- $window.NREUM.noticeError(exception);`
	`57`	`+ $window.NREUM.noticeError(exception, { url: $window.location.href });`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`// check if the config says we should log to the remote, and also if a remote endpoint was specified`
`@@ -149,7 +149,7 @@ loggingModule.factory(`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`if (sendToNewRelic && $window.NREUM && $window.NREUM.noticeError) {`
`152`		`- $window.NREUM.noticeError(message, {desc: desc});`
	`152`	`+ $window.NREUM.noticeError(message, { desc: desc, url: $window.location.href });`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`// check if the config says we should log to the remote, and also if a remote endpoint was specified`