dead_bro 0.2.32 → 0.2.34

data/lib/dead_bro.rb

@@ -30,6 +30,7 @@ module DeadBro
   autoload :MemoryDetails, "dead_bro/memory_details"
   autoload :Logger, "dead_bro/logger"
   autoload :WatchTracker, "dead_bro/watch_tracker"
+  autoload :Sanitizer, "dead_bro/sanitizer"
   begin
     require "dead_bro/railtie"
   rescue LoadError
@@ -37,6 +38,23 @@ module DeadBro
 
   class Error < StandardError; end
 
+  # What DeadBro code running inside the host's request or job — notification
+  # callbacks, middleware — rescues so it can never fail the host. Rails re-raises
+  # a notification listener's exception into the instrumented block, so a raise
+  # in a process_action callback turns the host's 200 into a 500.
+  #
+  # Deliberately not Exception: an asynchronous interrupt that happens to land
+  # while our code runs (Timeout, Rack::Timeout, Sidekiq::Shutdown, Interrupt)
+  # must still reach the host, or its timeout/shutdown handling silently stops
+  # working. These are the errors our own code can raise synchronously.
+  #
+  # One gap remains: Timeout.timeout(sec, SomeStandardError) delivers a
+  # StandardError, so it is swallowed if it fires mid-callback — as it would be
+  # by any `rescue => e` in the host's own code, on every Ruby. Plain
+  # Timeout.timeout(sec) is not affected. Rescuing Exception wouldn't close that
+  # gap; it would only open the ones above.
+  CONTAINED_ERRORS = [StandardError, ScriptError, SystemStackError].freeze
+
   # Returned by DeadBro.analyze. sql_queries is intentionally omitted from
   # inspect/to_s/pretty_print to avoid bloating console output.
   # Access it explicitly via .sql_queries.
@@ -338,6 +356,7 @@ module DeadBro
             end
 
             local_sql_queries << {duration_ms: duration_ms, sql: normalized_sql, query_type: query_type}
+          rescue *DeadBro::CONTAINED_ERRORS
           end
       end
     rescue

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dead_bro
 version: !ruby/object:Gem::Version
-  version: 0.2.32
+  version: 0.2.34
 platform: ruby
 authors:
 - Emanuel Comsa
@@ -18,6 +18,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - CHANGELOG.md
+- FEATURES.md
 - README.md
 - lib/dead_bro.rb
 - lib/dead_bro/allocation_source_sampler.rb
@@ -53,6 +54,7 @@ files:
 - lib/dead_bro/monitor.rb
 - lib/dead_bro/railtie.rb
 - lib/dead_bro/redis_subscriber.rb
+- lib/dead_bro/sanitizer.rb
 - lib/dead_bro/sql_subscriber.rb
 - lib/dead_bro/sql_tracking_middleware.rb
 - lib/dead_bro/subscriber.rb
@@ -80,7 +82,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.22
 specification_version: 4
 summary: Minimal APM for Rails apps.
 test_files: []