logging: Raven handler fixes

- pass logger name
- pass level not as a tag (this overwrites default "level" tag value in
Sentry UI instead of setting its second value)
- allow for passing additional data when logging exception

diff --git a/src/fnpjs/logging/handlers.js b/src/fnpjs/logging/handlers.js
index c5b6990..cc2eab1 100644 (file)
--- a/src/fnpjs/logging/handlers.js
+++ b/src/fnpjs/logging/handlers.js
@@ -10,16 +10,21 @@ return {
             return;
         }
 
             return;
         }
 

-        var ravenData = {};
+        var ravenData = {
+            level: record.level,
+            logger: record.loggerName,
+            tags: {}
+        };
+
+        Object.keys(record.data || {})
+            .filter(function(key) {return key !== 'exception';})
+            .forEach(function(key) {
+                ravenData.tags[key] = record.data[key];
+            });

 
         if(record.data.exception) {
 
         if(record.data.exception) {

-            window.Raven.captureException(record.data.exception);
+            window.Raven.captureException(record.data.exception, ravenData);

         } else {
         } else {

-            Object.keys(record.data || {}).forEach(function(key) {
-                ravenData[key] = record.data[key];
-            });
-            ravenData.tags = ravenData.tags || {};
-            ravenData.tags.level = record.level;

             window.Raven.captureMessage(record.message, ravenData);
         }
     }
             window.Raven.captureMessage(record.message, ravenData);
         }
     }