Skip to content

Commit

Permalink
[webui] Notify us about some routing exceptions
Browse files Browse the repository at this point in the history
There is only one general exception for missing files but there are
different causes for it. We don't really wan't to get notified about
people browsing to non-existing urls but we want to know about
rendering issues of our own application.
  • Loading branch information
hennevogel authored and coolo committed Nov 22, 2012
1 parent 706cc69 commit 746a935
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 15 deletions.
10 changes: 0 additions & 10 deletions src/webui/app/views/exception_notifier/_session.text.erb

This file was deleted.

21 changes: 16 additions & 5 deletions src/webui/config/initializers/hoptoad.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,21 @@
if CONFIG['errbit_api_key'].blank? || CONFIG['errbit_host'].blank?
config.development_environments = "production development test"
else
config.development_environments = "development test"
config.development_environments = "development test"
end
# We don't want to know about timeout errors, the api will tell us the real reason
config.ignore << Timeout::Error
# The api sometimes sends responses without a proper "Status:..." line (when it restarts?)
config.ignore << Net::HTTPBadResponse

config.ignore_only = %w{
ActiveRecord::RecordNotFound
ActionController::InvalidAuthenticityToken
CGI::Session::CookieStore::TamperedWithCookie
ActionController::UnknownAction
AbstractController::ActionNotFound
Timeout::Error
Net::HTTPBadResponse
}

config.ignore_by_filter do |exception_data|
true if exception_data[:class] == "ActionController::RoutingError" && exception_data[:message].includes("[GET]")
end

end

0 comments on commit 746a935

Please sign in to comment.