Enable opting out of patches

CHANGELOG.md

@@ -9,6 +9,7 @@
     config.rails.active_support_logger_subscription_items.delete("sql.active_record")
     config.rails.active_support_logger_subscription_items["foo"] = :bar
   ```
+- Enable opting out of patches [#2151](https://github.com/getsentry/sentry-ruby/pull/2151)
 
 ### Bug Fixes
 

sentry-ruby/lib/sentry-ruby.rb

@@ -75,30 +75,30 @@ def exception_locals_tp
     ##### Patch Registration #####
 
     # @!visibility private
-    def register_patch(patch = nil, target = nil, &block)
+    def register_patch(key, patch = nil, target = nil, &block)
       if patch && block
         raise ArgumentError.new("Please provide either a patch and its target OR a block, but not both")
       end
 
       if block
-        registered_patches << block
+        registered_patches[key] = block
       else
-        registered_patches << proc do
+        registered_patches[key] = proc do
           target.send(:prepend, patch) unless target.ancestors.include?(patch)
         end
       end
     end
 
     # @!visibility private
     def apply_patches(config)
-      registered_patches.each do |patch|
-        patch.call(config)
+      registered_patches.each do |key, patch|
+        patch.call(config) if config.enabled_patches.include?(key)
       end
     end
 
     # @!visibility private
     def registered_patches
-      @registered_patches ||= []
+      @registered_patches ||= {}
     end
 
     ##### Integrations #####

sentry-ruby/lib/sentry/configuration.rb

@@ -258,6 +258,11 @@ def capture_exception_frame_locals=(value)
     # @return [Float, nil]
     attr_reader :profiles_sample_rate
 
+    # Array of patches to apply.
+    # Default is {DEFAULT_PATCHES}
+    # @return [Array<Symbol>]
+    attr_accessor :enabled_patches
+
     # these are not config options
     # @!visibility private
     attr_reader :errors, :gem_specs
@@ -297,6 +302,8 @@ def capture_exception_frame_locals=(value)
 
     PROPAGATION_TARGETS_MATCH_ALL = /.*/.freeze
 
+    DEFAULT_PATCHES = %i(redis puma http)
+
     class << self
       # Post initialization callbacks are called at the end of initialization process
       # allowing extending the configuration of sentry-ruby by multiple extensions
@@ -340,6 +347,7 @@ def initialize
       self.server_name = server_name_from_env
       self.instrumenter = :sentry
       self.trace_propagation_targets = [PROPAGATION_TARGETS_MATCH_ALL]
+      self.enabled_patches = DEFAULT_PATCHES
 
       self.before_send = nil
       self.before_send_transaction = nil

sentry-ruby/lib/sentry/net/http.rb

@@ -99,4 +99,4 @@ def propagate_trace?(url, configuration)
   end
 end
 
-Sentry.register_patch(Sentry::Net::HTTP, Net::HTTP)
+Sentry.register_patch(:http, Sentry::Net::HTTP, Net::HTTP)

sentry-ruby/lib/sentry/puma.rb

@@ -29,4 +29,4 @@ def lowlevel_error(e, env, status=500)
   end
 end
 
-Sentry.register_patch(Sentry::Puma::Server, Puma::Server)
+Sentry.register_patch(:puma, Sentry::Puma::Server, Puma::Server)

sentry-ruby/lib/sentry/redis.rb

@@ -99,8 +99,10 @@ def call_pipelined(commands, redis_config)
 
 if defined?(::Redis::Client)
   if Gem::Version.new(::Redis::VERSION) < Gem::Version.new("5.0")
-    Sentry.register_patch(Sentry::Redis::OldClientPatch, ::Redis::Client)
+    Sentry.register_patch(:redis, Sentry::Redis::OldClientPatch, ::Redis::Client)
   elsif defined?(RedisClient)
-    RedisClient.register(Sentry::Redis::GlobalRedisInstrumentation)
+    Sentry.register_patch(:redis) do
+      RedisClient.register(Sentry::Redis::GlobalRedisInstrumentation)
+    end
   end
 end