Financial-Times / o-errors

Decoupled client-side error-logging
http://registry.origami.ft.com/components/o-errors
1 stars 2 forks source link

Throttle number of errors reported #28

Open kornelski opened 9 years ago

kornelski commented 9 years ago

Our previous implementation used to limit number of errors to maximum of 60 per minute. This (arbitrary) limit was meant to prevent overloading with error reports when an error happens in a loop.

For example if the app does:

for(var i=0; i < 9999; i++) try { foo.undefined(); } catch(err) { oErrors.report(err); }

we'd expect to get only 60 errors and drop the rest.

kornelski commented 9 years ago

This is a low priority for the webapp (for now we'll use the filter callback to reimplement it), but may be a good thing to have by default for all users.