1182 setup bugsnag (#1184)

* Add bugsnag gem

* Add bugsnag api key to credentials

* Add bugsnag initialize

* Reorder rescue in AppController

* Log handled exceptions

* Exclude authz errors from logging

* Avoid re-notifying an exception

* Update app/controllers/application_controller.rb

Co-authored-by: Justin <[email protected]>

---------

Co-authored-by: Justin <[email protected]>

Gemfile

@@ -121,6 +121,9 @@ gem "dry-initializer", "~> 3.1"
 # Catch unsafe migrations
 gem "strong_migrations", "~> 2.1"
 
+# Track errors in prod
+gem "bugsnag", "~> 6.27"
+
 group :development, :test, :staging do
   gem "faker"
 end

Gemfile.lock

@@ -123,6 +123,8 @@ GEM
       activemodel (>= 6.1)
     brakeman (6.2.2)
       racc
+    bugsnag (6.27.1)
+      concurrent-ruby (~> 1.0)
     builder (3.3.0)
     capybara (3.40.0)
       addressable
@@ -554,6 +556,7 @@ DEPENDENCIES
   bootstrap
   bootstrap_form (~> 5.4)
   brakeman
+  bugsnag (~> 6.27)
   capybara
   city-state (~> 1.1.0)
   cuprite

app/controllers/application_controller.rb

@@ -1,10 +1,17 @@
 class ApplicationController < ActionController::Base
   before_action :authenticate_user!
   verify_authorized unless: :devise_controller?
-
   before_action :set_current_user
   around_action :switch_locale
 
+  KNOWN_ERRORS = [ActionPolicy::Unauthorized]
+  rescue_from StandardError, with: :log_and_reraise
+  rescue_from ActionPolicy::Unauthorized do |ex|
+    flash[:alert] = t("errors.authorization_error")
+
+    redirect_back_or_to root_path
+  end
+
   def set_current_user
     Current.user = current_user
   end
@@ -14,9 +21,11 @@ def switch_locale(&action)
     I18n.with_locale(locale, &action)
   end
 
-  rescue_from ActionPolicy::Unauthorized do |ex|
-    flash[:alert] = t("errors.authorization_error")
-
-    redirect_back fallback_location: root_path
+  def log_and_reraise(error)
+    unless KNOWN_ERRORS.include?(error.class)
+      Bugsnag.notify(error)
+      error.instance_eval { def skip_bugsnag = true }
+    end
+    raise
   end
 end

config/credentials/production.yml.enc

@@ -1 +1 @@
-B8stKWMaJ/RVHL44hfyXskbZj+ioVbNZ0V4Jx8u6JSIhY0rbL7QDNqZGnguv1jR/OUN1DNeR6MvNP3nPAfXuSFW9Iq3T5wZgqOga5k4cB/wupr66FyBx+EkuE8Xp6ghDrCZHoxkAyMkly7WfXzpp1linj4Dt5YyU2V7cK7b6JA4JEa/1/8faHeNC2h4PwSHbQqoVyGk24yvu9vSn8/0q0Wp8gSwh8/1gPWva5Xa4S65CaMHgl9OqavtwZ7EpV9infVYoQd0cbhT9nK/EutMl9/n6hraxAir/XsiNJZvC7teDGfpWMw0gR3U8AbL9rFVSqXOfeDymjPM2GCf/mMe8eeAO9TKXQJcvcQ37fVOwqMgzmMGQA6+KcZJDY81JI9Dwz+u9X2+gh88f0rP7BkxTtTl7EbSTLqWhNCvPNE0lm252vJ2EVWcK6rNaYwr8gmFNmsF8jC7WbGnrZyM5TB68rgkoIgLfAZxo0W5Xhsahdv8ULF4QtkH+jbDD1EesR+bLp2GlWgl0G/EGRwYt0fUml8Vzp8hd2XqGLLndkA2NjSrnWgtJdjZ5/Qe1LDUrNLtXo0q1hIrBjlpNZZjYqA6wsEPiqSca92Lbpv9ib56FSWfzjgBJQtxpJYP3YUGak5TmJUomu+RRB2cHfCMTIKhrV4PmsOd8vROq0Rm8X6FotQ9eSWZ//ej1yZagW0vGpcSv4ubn9zr5tF+sABr1cCga98HYkgU+uSLtyV5eDfMvglT4FxQ4uIbAJjhe5jn+sXHHR179YrH+BEU3EnuSkquHJe/kXnRJHO4pUyragaWvlMEYPJjkhnlAyYkfcpRUI4Zu8g1AoOcjyzBzTui6qhC6hcNSxqf+vE5saNHA1ySIMO7GpkeOeftaxJ457Mz+RWeIkKUnC23uN4pXZs8eDSreUdrFQlPvd5E66j7tdWfuCPjMNHsU704kf463XxOstORJZauP/D9zvjiuc1kM/Iy52nRpNu6mRfCvOnNvfIcCcuWw/ZwxD2HM/uD35B0jTNXKMzMD+c55xrvyi4ZVA0qftHLr6uZmqyWuI7kE5AEV0Kc1Juo36IFQHhyY4cBxfWU=--Z8VjeW8okb1UWg+f--kNXS4gCxO58e0IwQlg6NDw==
+4ERfYFtT9kWCK++LezB5gW0pmP8hN+mI1HQ9+Hx9N5TWfR7eIsQFlCoKKFrM3RSe3p5ZSDL5YTdFBKIFscYa23Sx9sgb0dw8hLc1Q0ItD4sfVqnbZPctFrWnUIlLihLthz2O94rhzjk0sk0rer1KMaD3ZPggl7sAcGY8Osy+eRUVDygvRhX0nPDEpgRo1Wh5xzda3iDW5LaIjICBLvUzSjiXr9tp/HFKzprwoJtqqeO0H54L4TVRmocfbtVsyNl8w4CDOftloYH1KfvfnQzzPquEuIFFvw2WuverL3BZ04fnDpvL6xMWM4Vt0jtA+pzmZoYZo29T0LoX+33j8v4UUV2OddkxR7rUXYWdpE7/U3NFc5GDM7ynyRh+Jp7+nlnuYbHdw1wbPW54HwY+mlq9G/Out3hNo/nTcY171xl9xI7utMKD0wvIQIVqId5xoAT+hJosJXcbM24tCvjd4Q3zGeH0xEwvgmgCojcTwcCX7Jn9tqU1Iy3F4KUQpll/kPz32TmruZA3pPGgqWNfr96ogJ1qPaWvI+6R9ZGmbKAdrufLdm4//pZBMUkXlRBk89Sl0dSZliw3LfSyAIOsVjOkQfftU+c63iWZCQ5eUxoIPaDo4p8YkftTbUKZmk+pAgon2kau2CtLVF9FbjA+MFB5xVmPi+JchKDT+knWjPIHE0vg9ePBoU26S4HAu29JQnLnwEAfK1hdJkClPl9HTIdDaZ38Ld1b68MR96s9ZlLk2EARm9Pc8UEjtNXS2o+7t1tLZoVllXd+MrWPHjaoKUWMaSc2TnNzMtVnVfB+Bxi6qYDs0Blbjwk+mwWsMsnlDIF38Fbrkt8YyMfJ9CBeFaAzi1BJw54NJ784UluEo3QUc5T1WRan3H6Nb5dgBSq9VhWGV0QKSJEtc2IvmDrfE3qn8IaR3E7NZbV6UoeczwEwnTI9HuvrZ0WkNdNugx8E1Dk/c8ZVjLmtcsB2XlQtW4mtzWGj+k9ZUuRCXe032xQwJbgHtBHbtVLi4SbRvYyiU5m4JfORuaMMu/V9EZXPjLvAyWdRlVKDptu+CbPgbbG0GPMSOpx/XfV0QgolZ0kgUTemawSTeBDVnkvufwIaOfImfEt5U/QnMUYJyR0ZBpjAXNqaE+bb+qVe+yBOJIWY6SETlPfIhWDfVfCDm+RY--BTHcPE2kQRjxri/C--sSAa87XeJ0VAblR+MOmoBg==

config/initializers/bugsnag.rb

@@ -0,0 +1,7 @@
+if Rails.application.credentials.bugsnag_api_key.present?
+  Bugsnag.configure do |config|
+    config.api_key = Rails.application.credentials.bugsnag_api_key
+  end
+else
+  Bugsnag.configuration.logger = ::Logger.new("/dev/null")
+end