Cleanup rollbar args

wework · Nov 7, 2017 · 1ea3b8f · 1ea3b8f
1 parent 9ba9e0d
commit 1ea3b8f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/util/client/rollbarLogger.js b/src/util/client/rollbarLogger.js
@@ -43,13 +43,14 @@ export default function ClientRollbarLogger({ token, environment, codeVersion })
  */
 ClientRollbarLogger.prototype.write = function (data = {}) {
   const rollbarLevelName = bunyanLevelToRollbarLevelName(data.level);
-  const scopeData = omit(data, ['req', 'level']);
+  const scopeData = omit(data, ['err', 'level']);
+  const payload = Object.assign({ level: rollbarLevelName }, scopeData);
 
   // https://rollbar.com/docs/notifier/rollbar.js/#rollbarlog
   const logFn = global.Rollbar[rollbarLevelName];
   if (isFunction(logFn)) {
-    logFn.call(global.Rollbar, data.msg, data.err, scopeData);
+    logFn.call(global.Rollbar, data.msg, data.err, payload);
   } else {
-    global.Rollbar.error(data.msg, data.err, scopeData);
+    global.Rollbar.error(data.msg, data.err, payload);
   }
 };
diff --git a/src/util/server/rollbarLogger.js b/src/util/server/rollbarLogger.js
@@ -30,7 +30,7 @@ export default function ServerRollbarLogger({ token, codeVersion, environment })
  */
 ServerRollbarLogger.prototype.write = function (data = {}) {
   const rollbarLevelName = bunyanLevelToRollbarLevelName(data.level);
-  const scopeData = omit(data, ['req', 'level']);
+  const scopeData = omit(data, ['req', 'err', 'level']);
   const payload = Object.assign({ level: rollbarLevelName }, scopeData);
 
   // https://rollbar.com/docs/notifier/rollbar.js/#rollbarlog-1