How to not ignore default ignored exceptions about exception_notification

[smartinez87]

You should pass an empty array to the 'ignore exceptions' option, like this:

Whatever::Application.config.middleware.use ExceptionNotifier,
:email_prefix => "[Whatever] ",
:sender_address => %{"notifier" [email protected]},
:exception_recipients => %w{[email protected]},
:ignore_exceptions => []

[AlexandreK38]

Thanks, but I already tried this, but it keep not sending email for a RoutingError, that's why I asked :(
According to the code, it should but it is not working. I still don't know why.

[gtd]

Sanity check: did you check the logs?

[AlexandreK38]

Yes, in production.log I see the exceptions, I saw the call to the plugin when I have an error other that default a,d the mail is sent without problem. I can see the configuration which looks like ok :

• exception_notifier.options : {:sender_address=>""_" <_t>", :exception_recipients=>"[email protected]", :email_prefix=>"[_*] ", :sections=>["request", "session", "environment", "backtrace"], :ignore_exceptions=>[]}

But nothing happen with a RoutingError.

[aroc]

I'm having the same issue as AlexandreK. Passing an empty array does not make exception_notification send an email. My logs definitely show a 404 error took place.

[smartinez87]

Can you provide a failing test for this?
Passing an empty array does make sending the notification in some sample apps I have.

[smartinez87]

@Alexandrek @aroc any update on this issue? Have you tried on the release candidate?
Please provide a sample app / failing test if it keeps happening to you, I'm not being able to reproduce this.

[smartinez87]

Closing this issue. Feel free to re-open it if you're still facing it.

[why-el]

This is still the case for rails 3.2.13. Passing an empty array to ignore_exceptions won't send an email for the errors that are ignored by default, such as RecordNotFound and RoutingError.

[jacobjlevine]

Hey @smartinez87, this is definitely still an issue. I think the problem is the below method. It makes it impossible to ever get notifications for the default ignored exceptions.

def ignored_exception?(ignore_array, exception)
  (Array(ignored_exceptions) + Array(ignore_array)).map(&:to_s).include?(exception.class.name)
end

[shanecav84]

I added ::ExceptionNotifier.ignored_exceptions = [] after my ::ExceptionNotification config to get this to work.