Raising rescued exceptions in Development mode about honeybadger-ruby HOT 3 CLOSED

Thanks for the suggestion! Unfortunately I see a few problems with this. Many people have logic in their code which comes after calling notify_honeybadger, so causing this to re-raise would seriously break some projects in development. Also, it's possible to call this method outside of a rescue block, and re-raising there would cause unexpected exceptions to be raised.

If you want to use this everywhere, I'd either override the notify_honeybadger, or build your own method which has the modified behavior:

def report_exception(ex)
  if Rails.env.development?
    raise ex  # or logger.error(ex) 
  else
    notify_honeybadger(ex)
  end
end

from honeybadger-ruby.

You are right that re-raising the exception might not be such a good idea since the code after that would not be executed in development. But then don't you think it would be at least a good idea to log the error in the console so we know something is happening? Or maybe have a global config to determine what should be the behaviour? Maybe a block of code like you have for the async config. Right now the only choices are to report or ignore the exception in development.

config.development_notify do |notice|  
  Rails.logger.error(ex) # The actual behaviour is set by the user in Honeybadger.configure
end

I could also implement a report_exception as you mention but that would only work in controllers and not my lib and model classes. I guess I would then need another report_exception method for those and suddenly it gets a bit messy for something that should probably handled directly in the gem!

from honeybadger-ruby.

Hey Pierre, sorry for not replying sooner! I get what you're saying, but I'm still not convinced this is something which belongs in the Honeybadger gem. Honeybadger is providing notification of errors, but it's still up to the user to handle the errors themselves. In your case I recommend creating your own error handling abstraction which will both notify Honeybadger in production and log in development (kind of like the report_exception method I mentioned earlier).

from honeybadger-ruby.