feat(webhook status): log unhandled suckerpunch errors

mattermack · Sep 6, 2017 · 4cc779d · 4cc779d
1 parent c9272dc
commit 4cc779d
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/lib/pact_broker/app.rb b/lib/pact_broker/app.rb
@@ -45,6 +45,7 @@ def post_configure
       PactBroker.logger = configuration.logger
       SuckerPunch.logger = configuration.logger
       configure_database_connection
+      configure_sucker_punch
     end
 
     def migrate_database
@@ -119,6 +120,12 @@ def build_diagnostic
       builder
     end
 
+    def configure_sucker_punch
+      SuckerPunch.exception_handler = -> (ex, klass, args) do
+        PactBroker.log_error(ex, "Unhandled Suckerpunch error for #{klass}.perform(#{args.inspect})")
+      end
+    end
+
     def running_app
       @running_app ||= begin
         apps = @cascade_apps