ref: use console.warn for alerts and store them in Set

getsentry · May 8, 2018 · f56d9f0 · f56d9f0
1 parent 381dd6f
commit f56d9f0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -2,6 +2,7 @@
   "env": {
     "node": true,
     "mocha": true,
+    "es6": true
   },
   "rules": {
     "block-scoped-var": 2,

diff --git a/lib/utils.js b/lib/utils.js
@@ -15,7 +15,7 @@ var protocolMap = {
   https: 443
 };
 
-var consoleAlerts = {};
+var consoleAlerts = new Set();
 
 // Default Node.js REPL depth
 var MAX_SERIALIZE_EXCEPTION_DEPTH = 3;
@@ -125,14 +125,14 @@ module.exports.disableConsoleAlerts = function disableConsoleAlerts() {
 
 module.exports.consoleAlert = function consoleAlert(msg) {
   if (consoleAlerts) {
-    console.log('raven@' + ravenVersion + ' alert: ' + msg);
+    console.warn('raven@' + ravenVersion + ' alert: ' + msg);
   }
 };
 
 module.exports.consoleAlertOnce = function consoleAlertOnce(msg) {
-  if (consoleAlerts && !(msg in consoleAlerts)) {
-    consoleAlerts[msg] = true;
-    console.log('raven@' + ravenVersion + ' alert: ' + msg);
+  if (consoleAlerts && !consoleAlerts.has(msg)) {
+    consoleAlerts.add(msg);
+    console.warn('raven@' + ravenVersion + ' alert: ' + msg);
   }
 };