error governor fix

Author: alexk-blackops

lib/error.js

@@ -15,13 +15,14 @@ module.exports = {
     checkErrorLimitMessage : function checkErrorLimitMessage(ex) {
         var d = new Date(),
             min = d.getFullYear().toString() + d.getMonth().toString() + d.getDate().toString()
-                + d.getHours().toString() + d.getMinutes().toString(),
-            key = ex.Error.Message + ex.Error.ErrorType + ex.Error.SourceMethod;
+                + d.getHours().toString() + d.getMinutes().toString();
 
         if (!ex) {
             return true;
         }
 
+        key = ex.Error.Message + ex.Error.ErrorType + ex.Error.SourceMethod;
+
         if (errorStorage[min]) {
             if (errorStorage[min][key]) {
                 errorStorage[min][key] += 1;

lib/logger.js

@@ -89,8 +89,11 @@ module.exports.methods = {
             if (level.toLowerCase() === 'error') {
                 data = helpers.parseMeta(meta, true);
                 // if duplicate errors cap per minute isn't exceeded and error object is passed, create an exception
-                if (data.ex && error.checkErrorLimitMessage(data.ex)) {
+                if (data.ex) {
                     messageObject.Ex = error.formatEx(data.ex, req);
+                    if (!error.checkErrorLimitMessage(messageObject.Ex)) {
+                        delete messageObject.Ex;
+                    }
                 }
             } else {
                 data = helpers.parseMeta(meta);
@@ -102,8 +105,12 @@ module.exports.methods = {
         }
 
         // if error object isn't passed with message & duplicate errors cap per minute isn't exceeded, create a string exception
-        if (level.toLowerCase() === 'error' && !messageObject.Ex && error.checkErrorLimitMessage(data.ex)) {
+        if (level.toLowerCase() === 'error' && !messageObject.Ex) {
             messageObject.Ex = error.formatEx(err, null, msg);
+
+            if (!error.checkErrorLimitMessage(messageObject.Ex)) {
+                delete messageObject.Ex;
+            }
         }
 
         storage.push(messageObject);