From: Aleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Date: Mon, 30 Dec 2013 12:57:56 +0000 (+0100)
Subject: logging: Raven handler fixes
X-Git-Url: https://git.mdrn.pl/fnpeditor.git/commitdiff_plain/27ae526d5467db76208193e2aa1e80ad69d164d4?ds=inline

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
--- a/src/fnpjs/logging/handlers.js
+++ b/src/fnpjs/logging/handlers.js
@@ -10,16 +10,21 @@ 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) {
-            window.Raven.captureException(record.data.exception);
+            window.Raven.captureException(record.data.exception, ravenData);
         } 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);
         }
     }