feat: update puma and autotuner plugins (#644)

* feat: update puma and autotuner plugins

This allows the puma and autotuner to emit an event for stats, or
alternavitely aggregate the stats to save on data.

Disable stats events for puma and autotuner:
```
puma:
  insights:
    events: false
autotuner:
  insights:
    events: false
```

Note: The autotuner plugin will still event the `report.autotuner` event
regardless of the above setting.

Enable aggregated metics for puma and autotuner.
```
puma:
  insights:
    metrics: true
autotuner:
  insights:
    metrics: true
```

* Fix test for changes in Sinatra

Author: roelbondoc

lib/honeybadger/config/defaults.rb

@@ -491,6 +491,31 @@ class Boolean; end
         description: "Number of seconds between registry flushes.",
         default: 60,
         type: Integer
+      },
+      :'puma.insights.events' => {
+        description: 'Enable automatic event capturing for Puma stats.',
+        default: true,
+        type: Boolean
+      },
+      :'puma.insights.metrics' => {
+        description: 'Enable automatic metric data aggregation for Puma stats.',
+        default: false,
+        type: Boolean
+      },
+      :'puma.insights.collection_interval' => {
+        description: 'The frequency in which the Honeybadger gem will collect Puma stats.',
+        default: 1,
+        type: Integer
+      },
+      :'autotuner.insights.events' => {
+        description: 'Enable automatic event capturing for Autotuner stats.',
+        default: true,
+        type: Boolean
+      },
+      :'autotuner.insights.metrics' => {
+        description: 'Enable automatic metric data aggregation for Autotuner stats.',
+        default: false,
+        type: Boolean
       }
     }.freeze
 

lib/honeybadger/plugins/autotuner.rb

@@ -10,17 +10,20 @@ module Autotuner
         execution do
           singleton_class.include(Honeybadger::InstrumentationHelper)
 
-          ::Autotuner.enabled = true
-
           ::Autotuner.reporter = proc do |report|
             Honeybadger.event("report.autotuner", report: report.to_s)
           end
 
-          metric_source 'autotuner'
-
           ::Autotuner.metrics_reporter = proc do |metrics|
-            metrics.each do |key, val|
-              gauge key, ->{ val }
+            if config.load_plugin_insights_events?(:autotuner)
+              Honeybadger.event('stats.autotuner', metrics)
+            end
+
+            if config.load_plugin_insights_metrics?(:autotuner)
+              metric_source 'autotuner'
+              metrics.each do |key, val|
+                gauge key, ->{ val }
+              end
             end
           end
         end

lib/puma/plugin/honeybadger.rb

@@ -12,7 +12,7 @@ def start(launcher)
         in_background do
           loop do
             puma_plugin.record
-            sleep 1
+            sleep [::Honeybadger.config.collection_interval(:puma).to_i, 1].max
           end
         end
       end
@@ -35,8 +35,14 @@ def record
     end
 
     def record_puma_stats(stats, context={})
-      STATS_KEYS.each do |stat|
-        gauge stat, context, ->{ stats[stat] } if stats[stat]
+      if Honeybadger.config.load_plugin_insights_events?(:puma)
+        Honeybadger.event('stats.puma', context.merge(stats))
+      end
+
+      if Honeybadger.config.load_plugin_insights_metrics?(:puma)
+        STATS_KEYS.each do |stat|
+          gauge stat, context, ->{ stats[stat] } if stats[stat]
+        end
       end
     end
   end

spec/fixtures/sinatra/app.rb

@@ -1,6 +1,7 @@
 require 'sinatra/base'
 
 class SinatraApp < Sinatra::Base
+  set :host_authorization, { permitted_hosts: [] }
   set :show_exceptions, false
   set :honeybadger_api_key, 'gem testing'