perf: don't insert middleware at all if they've been disabled

honeybadger-io / honeybadger-ruby

Ruby gem for reporting errors to honeybadger.io

https://docs.honeybadger.io/lib/ruby/

MIT License

250 stars 146 forks source link

Closed bradleypriest closed 5 months ago

bradleypriest commented 5 months ago

Following up on my comment from 7 years ago.

If I've disabled the middlewares I would like to not see them in rails middleware

stympy commented 5 months ago

I approved this before I saw that the tests were failing, but once the tests pass, I think this is ready to go. :)

bradleypriest commented 5 months ago

I had a bit of a play around and it looks like the Agent and Config should be instantiated at this point