can set maxErrors config option

Author: MaxBittker

src/raven.js

@@ -57,6 +57,7 @@ function Raven() {
     this._keypressTimeout;
     this._location = _window.location;
     this._lastHref = this._location && this._location.href;
+    this._sentErrors = 0;
 
     for (var method in this._originalConsole) {  // eslint-disable-line guard-for-in
       this._originalConsoleMethods[method] = this._originalConsole[method];
@@ -770,6 +771,9 @@ Raven.prototype = {
         var parsedTo = parseUrl(to);
         var parsedFrom = parseUrl(from);
 
+        //refresh max error count
+        this._sentErrors = 0;
+
         // because onpopstate only tells you the "new" (to) value of location.href, and
         // not the previous (from) value, we need to track the value of the current URL
         // state ourselves
@@ -1059,6 +1063,9 @@ Raven.prototype = {
                 return function (/* state, title, url */) {
                     var url = arguments.length > 2 ? arguments[2] : undefined;
 
+                    //refresh max error count
+                    self._sentErrors = 0;
+
                     // url argument is optional
                     if (url) {
                         // coerce to string (this is what pushState does)
@@ -1352,7 +1359,12 @@ Raven.prototype = {
         if (isFunction(globalOptions.shouldSendCallback) && !globalOptions.shouldSendCallback(data)) {
             return;
         }
+        //Check if the request should be filtered due to errors per page
+        if (globalOptions.maxErrors && this._sentErrors >= globalOptions.maxErrors){
+           return;
+        }
 
+        this._sentErrors++; //failed errors count towards maxErrors
         this._sendProcessedPayload(data);
     },
 

test/raven.test.js

@@ -2218,6 +2218,60 @@ describe('Raven (public API)', function() {
         });
     });
 
+    describe('maxErrors config', function(){
+
+      it('allows many errors when maxErrors is undefined', function () {
+        var stub = this.sinon.stub(Raven,'_sendProcessedPayload')
+        this.sinon.spy(stub)
+
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+
+
+        assert.equal(Raven._sendProcessedPayload.callCount, 5);
+      });
+
+      it('should only allow up to maxErrors requests', function () {
+        var stub = this.sinon.stub(Raven,'_sendProcessedPayload')
+        this.sinon.spy(stub)
+
+        Raven._globalOptions.maxErrors = 3;
+
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+
+        assert.equal(Raven._sendProcessedPayload.callCount, 3);
+      });
+
+      it('should reset maxErrors on SPA page change', function () {
+        var stub = this.sinon.stub(Raven,'_sendProcessedPayload')
+        this.sinon.spy(stub)
+
+        Raven._globalOptions.maxErrors = 3;
+
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+
+        assert.equal(Raven._sendProcessedPayload.callCount, 3);
+
+        Raven._captureUrlChange('/foo', '/bar');
+
+        Raven.captureException(new Error('foo'))
+        Raven.captureException(new Error('foo'))
+
+        assert.equal(Raven._sendProcessedPayload.callCount, 5);
+      });
+
+    });
+
     describe('._captureUrlChange', function () {
         it('should create a new breadcrumb from its "from" and "to" arguments', function () {
             Raven._breadcrumbs = [];