removes 300-500 status codes from the default statsd

ansble / monument

event based http server for nodejs

http://monument.ansble.com

MIT License

34 stars 45 forks source link

removes 300-500 status codes from the default statsd #599

Closed designfrontier closed 6 years ago

designfrontier commented 6 years ago

Timers are now only sent by default for 100-2xx status codes this should make the stats more useful and more searchable.

coveralls commented 6 years ago

Coverage increased (+0.004%) to 98.218% when pulling afd157235bb6a812b2e19486b2382af0d3cfb293 on feature/limit-statsd-timers into 65bba9c1ff8bcd7963581eb5460780f2af5de3ab on Paris–Brussels-5.x.x.

designfrontier commented 6 years ago

closes #598

designfrontier commented 6 years ago

The failing test is throwing correctly... but the new error in node 10 is causing some issues