Merge pull request #13 from talis/ELE-3189-log-filter

ELE-3189 Callback check to allow errors to be filtered
techfromsage · Mar 16, 2020 · 1e6557f · 1e6557f
2 parents cdcb159 + 6f7f500
commit 1e6557f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
     "name": "app-logger-angular",
-    "version": "2.1.8",
+    "version": "2.2.0",
     "main": "./js/logging.js",
     "description": "Client side logging sent to the server",
     "repository": {

diff --git a/js/logging.js b/js/logging.js
@@ -49,6 +49,10 @@ loggingModule.factory(
                 $log.error.apply($log, arguments);
             }
 
+            if($window.talisLogger.onBeforeLogError && !$window.talisLogger.onBeforeLogError(exception)) {
+                return false;
+            }
+
             if (LOGGING_CONFIG.FORWARD_TO_NEWRELIC && $window.NREUM && $window.NREUM.noticeError) {
                 $window.NREUM.noticeError(exception);
             }
@@ -95,6 +99,7 @@ loggingModule.factory(
         var arrLoggingLevels = ['trace', 'debug', 'info', 'warn', 'error'];
         var loggingThreshold = LOGGING_CONFIG.LOGGING_THRESHOLD || 'info';
         var iLoggingThreshold = arrLoggingLevels.indexOf(loggingThreshold);
+        $window.talisLogger = {};
 
         /*
          * If we've told applicationLoggingService to override the logging threshold set in config then also pass
@@ -139,6 +144,10 @@ loggingModule.factory(
                 }
             }
 
+            if($window.talisLogger.onBeforeLogError && !$window.talisLogger.onBeforeLogError(message)) {
+                return false;
+            }
+
             if (sendToNewRelic && $window.NREUM && $window.NREUM.noticeError) {
                 $window.NREUM.noticeError(message, {desc: desc});
             }