contrast-agent 4.9.0 → 4.12.0

data/lib/contrast/agent/version.rb

@@ -3,6 +3,6 @@
 
 module Contrast
   module Agent
-    VERSION = '4.9.0'
+    VERSION = '4.12.0'
   end
 end

data/lib/contrast/agent.rb

@@ -20,7 +20,6 @@ require 'contrast/extension/delegator'
 require 'contrast/extension/inventory'
 require 'contrast/extension/module'
 require 'contrast/extension/protect'
-require 'contrast/extension/protect/kernel'
 
 require 'contrast/utils/object_share'
 require 'contrast/utils/string_utils'

data/lib/contrast/api/communication/messaging_queue.rb

@@ -11,10 +11,9 @@ module Contrast
       class MessagingQueue < Contrast::Agent::WorkerThread
         include Contrast::Components::Logger::InstanceMethods
 
-        attr_reader :queue, :speedracer
+        attr_reader :speedracer
 
         def initialize
-          @queue = Queue.new
           @speedracer = Contrast::Api::Communication::Speedracer.new
           super
         end
@@ -28,6 +27,10 @@ module Contrast
           speedracer.return_response(event)
         end
 
+        def queue
+          @_queue ||= Queue.new
+        end
+
         # Use this to add a message to the queue and process the response internally
         def send_event_eventually event
           if ::Contrast::AGENT.disabled?
@@ -42,7 +45,6 @@ module Contrast
           speedracer.ensure_startup!
           return if running?
 
-          @queue ||= Queue.new
           @_thread = Contrast::Agent::Thread.new do
             loop do
               event = queue.pop
@@ -58,13 +60,17 @@ module Contrast
           logger.debug('Started background sending thread.')
         end
 
+        def delete_queue!
+          @_queue&.clear
+          @_queue&.close
+          @_queue = nil
+        end
+
         def stop!
           return unless running?
 
           super
-          @queue&.clear
-          @queue&.close
-          @queue = nil
+          delete_queue!
         end
       end
     end

data/lib/contrast/api/communication/service_lifecycle.rb

@@ -6,10 +6,13 @@ require 'contrast/components/logger'
 module Contrast
   module Api
     module Communication
-      # Handles local service startup
+      # Handles local service startup. As this should only ever be invoked by the Speedracer class, which includes
+      # this, all methods here are private.
       module ServiceLifecycle
         include Contrast::Components::Logger::InstanceMethods
 
+        private
+
         def attempt_local_service_startup
           zombie_check
           service_starter_thread.join(5)

data/lib/contrast/api/communication/socket_client.rb

@@ -37,8 +37,7 @@ module Contrast
           log_connection
           if ::Contrast::CONTRAST_SERVICE.use_tcp?
             Contrast::Api::Communication::TcpSocket.new(
-              ::Contrast::CONTRAST_SERVICE.host, ::Contrast::CONTRAST_SERVICE.port
-            )
+                ::Contrast::CONTRAST_SERVICE.host, ::Contrast::CONTRAST_SERVICE.port)
           else
             Contrast::Api::Communication::UnixSocket.new(::Contrast::CONTRAST_SERVICE.socket_path)
           end
@@ -61,8 +60,9 @@ module Contrast
 
           # Or something is not set.
           logger.warn(
-            log_connection_error_msg, host: ::Contrast::CONTRAST_SERVICE.host, port: ::Contrast::CONTRAST_SERVICE.port
-          )
+              log_connection_error_msg,
+              host: ::Contrast::CONTRAST_SERVICE.host,
+              port: ::Contrast::CONTRAST_SERVICE.port)
         end
 
         # If our connection isn't built properly, we need to warn the user. This builds out the context specific

data/lib/contrast/api/decorators/agent_startup.rb

@@ -41,11 +41,11 @@ module Contrast
           #
           # @param msg [Contrast::Api::Dtm::AgentStartup]
           def config! msg
-            msg.version          = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.server.version
-            msg.environment      = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.server.environment
-            msg.server_tags      = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.server.tags
+            msg.version      = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.server.version
+            msg.server_tags  = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.server.tags
+            msg.library_tags = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.inventory.tags
+            msg.environment  = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.server.environment
             msg.application_tags = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.tags
-            msg.library_tags     = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.inventory.tags
           end
         end
       end

data/lib/contrast/api/decorators/application_startup.rb

@@ -24,11 +24,12 @@ module Contrast
           # @return [Contrast::Api::Dtm::ApplicationCreate]
           def build
             msg = new
-            msg.app_version = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.version.to_s
-            msg.code        = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.code
-            msg.group       = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.group
-            msg.metadata    = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.metadata
-            msg.mode        = Contrast::Api::Dtm::InstrumentationMode.build
+            msg.code     = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.code
+            msg.group    = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.group
+            msg.metadata = Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.metadata
+            msg.mode     = Contrast::Api::Dtm::InstrumentationMode.build
+            msg.app_version =
+                Contrast::Utils::StringUtils.protobuf_format ::Contrast::CONFIG.root.application.version.to_s # rubocop:disable Layout/AssignmentIndentation Layout/FirstArgumentIndentation:
             session!(msg)
             msg
           end

data/lib/contrast/api/decorators/route_coverage.rb

@@ -46,7 +46,7 @@ module Contrast
           #
           # @param controller [::Sinatra::Base] the route's final controller.
           # @param method [String] GET, PUT, POST, etc...
-          # @param method [::Mustermann::Sinatra]  the pattern that was matched in routing.
+          # @param pattern [::Mustermann::Sinatra]  the pattern that was matched in routing.
           # @param url [String, nil] use url from string instead matched pattern.
           # @return [Contrast::Api::Dtm::RouteCoverage]
           def from_sinatra_route controller, method, pattern, url = nil
@@ -59,6 +59,29 @@ module Contrast
             msg.url = Contrast::Utils::StringUtils.force_utf8(safe_url)
             msg
           end
+
+          # Convert Grape route data to dtm message.
+          #
+          # @param controller [::Grape::API] the route's final controller.
+          # @param method [String] GET, PUT, POST, etc...
+          # @param url [String, nil] use url from string instead matched pattern.
+          # @param pattern [String, Grape::Router::Route] the pattern that was matched in routing.
+          # @return [Contrast::Api::Dtm::RouteCoverage]
+          def from_grape_controller controller, method, pattern, url = nil
+            if pattern.cs__is_a?(Grape::Router::Route)
+              safe_pattern = pattern.pattern&.path&.to_s
+              safe_url = source_or_string(url || safe_pattern)
+            else
+              safe_pattern = source_or_string(pattern)
+              safe_url = source_or_string(url || pattern)
+            end
+
+            msg = new
+            msg.route = "#{ controller }##{ method } #{ safe_pattern }"
+            msg.verb = Contrast::Utils::StringUtils.force_utf8(method)
+            msg.url = Contrast::Utils::StringUtils.force_utf8(safe_url)
+            msg
+          end
         end
       end
     end

data/lib/contrast/components/agent.rb

@@ -54,7 +54,9 @@ module Contrast
         end
 
         def interpolation_enabled?
-          @_interpolation_enabled = !false?(::Contrast::CONFIG.root.agent.ruby.interpolate) if @_interpolation_enabled.nil?
+          if @_interpolation_enabled.nil?
+            @_interpolation_enabled = !false?(::Contrast::CONFIG.root.agent.ruby.interpolate)
+          end
           @_interpolation_enabled
         end
 
@@ -69,7 +71,8 @@ module Contrast
               status:
                 ::Contrast::CONFIG.root.agent.ruby.exceptions.override_status || 403,
               message:
-                ::Contrast::CONFIG.root.agent.ruby.exceptions.override_message || Contrast::Utils::ObjectShare::OVERRIDE_MESSAGE
+                ::Contrast::CONFIG.root.agent.ruby.exceptions.override_message ||
+                    Contrast::Utils::ObjectShare::OVERRIDE_MESSAGE
           }
         end
 

data/lib/contrast/components/assess.rb

@@ -5,7 +5,6 @@ require 'contrast/components/base'
 require 'contrast/components/config'
 require 'contrast/components/settings'
 
-
 module Contrast
   module Components
     module Assess
@@ -78,7 +77,9 @@ module Contrast
         end
 
         def track_frozen_sources?
-          @_track_frozen_sources = !false?(::Contrast::CONFIG.root.agent.ruby.track_frozen_sources) if @_track_frozen_sources.nil?
+          if @_track_frozen_sources.nil?
+            @_track_frozen_sources = !false?(::Contrast::CONFIG.root.agent.ruby.track_frozen_sources)
+          end
           @_track_frozen_sources
         end
 
@@ -87,13 +88,22 @@ module Contrast
           @_require_scan
         end
 
+        def require_dynamic_sources?
+          if @_require_dynamic_sources.nil?
+            @_require_dynamic_sources = !false?(::Contrast::CONFIG.root.assess.enable_dynamic_sources)
+          end
+          @_require_dynamic_sources
+        end
+
         def tags
           ::Contrast::CONFIG.root.assess&.tags
         end
 
         def disabled_rules
           # TODO: RUBY-903
-          ::Contrast::CONFIG.root.assess&.rules&.disabled_rules || ::Contrast::SETTINGS.assess_state.disabled_assess_rules || []
+          ::Contrast::CONFIG.root.assess&.rules&.disabled_rules ||
+              ::Contrast::SETTINGS.assess_state.disabled_assess_rules ||
+              []
         end
 
         private

data/lib/contrast/components/base.rb

@@ -18,7 +18,7 @@ module Contrast
         return true if config_param == false
         return false unless config_param.cs__is_a?(String)
 
-        Contrast::Utils::ObjectShare::FALSE.casecmp?(config_param)
+        config_param.downcase == Contrast::Utils::ObjectShare::FALSE
       end
 
       # use this to determine if the configuration value is literally boolean
@@ -33,7 +33,7 @@ module Contrast
         return true if config_param == true
         return false unless config_param.cs__is_a?(String)
 
-        Contrast::Utils::ObjectShare::TRUE.casecmp?(config_param)
+        config_param.downcase == Contrast::Utils::ObjectShare::TRUE
       end
     end
   end

data/lib/contrast/components/config.rb

@@ -44,6 +44,7 @@ module Contrast
 
         def valid?
           @_valid = validate(log: false) if @_valid.nil?
+          @_valid
         end
 
         def invalid?

data/lib/contrast/components/contrast_service.rb

@@ -32,11 +32,13 @@ module Contrast
         end
 
         def host
-          @_host ||= (::Contrast::CONFIG.root.agent.service.host || Contrast::Config::ServiceConfiguration::DEFAULT_HOST).to_s
+          @_host ||=
+            (::Contrast::CONFIG.root.agent.service.host || Contrast::Config::ServiceConfiguration::DEFAULT_HOST).to_s
         end
 
         def port
-          @_port ||= (::Contrast::CONFIG.root.agent.service.port || Contrast::Config::ServiceConfiguration::DEFAULT_PORT).to_i
+          @_port ||=
+            (::Contrast::CONFIG.root.agent.service.port || Contrast::Config::ServiceConfiguration::DEFAULT_PORT).to_i
         end
 
         def socket_path

data/lib/contrast/components/logger.rb

@@ -6,20 +6,25 @@ require 'contrast/components/base'
 
 module Contrast
   module Components
-    module Logger
-      module InstanceMethods #:nodoc:
+    module Logger # :nodoc:
+      module InstanceMethods # :nodoc:
         def logger
           Contrast::Logger::Log.instance.logger
         end
+
+        def add_trace_perf_logging_for sym, custom_message = nil
+          logger.add_trace_perf_logging(self, sym, custom_message)
+        end
+      end
+
+      class << self
+        def add_trace_log_timing_for clazz, method_name, custom_message = nil
+          Contrast::Logger::Log.instance.add_method_to_trace_timing(clazz, method_name, custom_message)
+        end
       end
-      ClassMethods = InstanceMethods
 
-      # A wrapper build around the Common Agent Configuration project to allow
-      # for access of the values contained in its
-      # parent_configuration_spec.yaml.
-      # Specifically, this allows for querying the state of the Agent Logger.
       class Interface
-        include Contrast::Components::ComponentBase
+        include InstanceMethods
       end
     end
   end

data/lib/contrast/components/scope.rb

@@ -7,12 +7,9 @@ require 'contrast/agent/scope'
 
 # This is the Scope component.
 #
-# It tracks /Contrast/ scope.  That is, "are we currently doing assess
-# or protect stuff within a patched method?" -- this is how we avoid doing
-# Contrast stuff on Contrast code.
-#
-# Separately from this component, there is also require scope, which is an
-# optimization on how we implement patching to `require`.
+# It tracks /Contrast/ scope.  That is, "are we currently doing assess or protect stuff within a patched method?" --
+# this is how we avoid doing Contrast stuff on Contrast code or creating infinite loops -- or "are we in some other
+# execution context for which we need to special case?".
 module Contrast
   module Components
     module Scope # :nodoc:
@@ -25,10 +22,8 @@ module Contrast
           EXECUTION_CONTEXT[Fiber.current] = Contrast::Agent::Scope.new
         end
 
-        # This returns the scope governing the current execution context.
-        # Use this sparingly, preferring the instance & class methods to
-        # access and query scope, rather than interacting with the scope
-        # object directly.
+        # This returns the scope governing the current execution context. Use this sparingly, preferring the instance
+        # & class methods to access and query scope, rather than interacting with the scope object directly.
         def scope_for_current_ec
           MONITOR.synchronize do
             return EXECUTION_CONTEXT[Fiber.current] ||= Contrast::Agent::Scope.new
@@ -37,9 +32,7 @@ module Contrast
       end
 
       module InstanceMethods # :nodoc:
-        # For each instance method on a scope, define a forwarder
-        # to the scope on the current execution context's scope.
-
+        # For each instance method on a scope, define a forwarder to the scope on the current execution context's scope.
         def scope_for_current_ec
           MONITOR.synchronize do
             return EXECUTION_CONTEXT[Fiber.current] ||= Contrast::Agent::Scope.new
@@ -118,24 +111,12 @@ module Contrast
         ensure
           scope_for_current_ec.exit_split_scope!
         end
-
-        # TODO: RUBY-572
-        #
-        # Current behavior is to no-op if we're not "in a request context".
-        # Our C functions were previously checking to see if we had a scope, because
-        # scope was tacked on to a request context -- so "we have a scope, therefore,
-        # we have a request context."  We've decoupled scopes from request contexts,
-        # so now it checks "do we have a request context."
-        # RUBY-290 should remove all of that, including this method.
-        def in_request_context?
-          !!Contrast::Agent::REQUEST_TRACKER.current
-        end
       end
 
       def self.sweep_dead_ecs
-        # TODO: RUBY-571, #sweep_dead_ecs compensates for a lack of weak tables
-        # 'ec' for execution context.  in this case, it's a Fiber.
-        # Threads rely on Fibers, so two birds, one stone.
+        # TODO: RUBY-534, #sweep_dead_ecs compensates for a lack of weak tables. when we can use WeakRef, we should
+        #   investigate removing this call and instead use the WeakRef for the Execution Context's Keys or using our
+        #   Finalizers Hash for Fibers
         MONITOR.synchronize do
           EXECUTION_CONTEXT.delete_if do |ec, _scope|
             !ec.alive?

data/lib/contrast/config/assess_configuration.rb

@@ -10,6 +10,7 @@ module Contrast
           tags: EMPTY_VALUE,
           enable: EMPTY_VALUE,
           enable_scan_response: Contrast::Config::DefaultValue.new('true'),
+          enable_dynamic_sources: Contrast::Config::DefaultValue.new('true'),
           sampling: Contrast::Config::SamplingConfiguration,
           rules: Contrast::Config::AssessRulesConfiguration,
           stacktraces: Contrast::Config::DefaultValue.new('ALL')

data/lib/contrast/config/base_configuration.rb

@@ -12,7 +12,7 @@ module Contrast
     class BaseConfiguration
       extend Forwardable
 
-      BOOLEANS = [true, false].cs__freeze
+      STRING_BOOLEANS = %w[false true].cs__freeze
 
       attr_reader :map
 
@@ -73,8 +73,18 @@ module Contrast
                           spec_value.new(user_provided_value)
                         elsif spec_value.is_a?(Contrast::Config::DefaultValue) && user_provided_value == EMPTY_VALUE
                           spec_value.value
-                        elsif BOOLEANS.include?(user_provided_value)
-                          user_provided_value.to_s
+                        elsif user_provided_value.cs__is_a?(String)
+                          value = user_provided_value.downcase
+                          # converts string values to 'true' => true or 'false' => false
+                          case value
+                          when STRING_BOOLEANS[1]
+                            true
+                          when STRING_BOOLEANS[0]
+                            false
+                          else
+                            # returns non boolean string values
+                            user_provided_value
+                          end
                         else
                           user_provided_value
                         end
@@ -95,9 +105,7 @@ module Contrast
 
       def define_setter str_key
         define_singleton_method "#{ str_key }=".to_sym do |new_value|
-          boolean_value = new_value == true
-          boolean_value ||= new_value == false
-          @map[str_key] = boolean_value ? new_value.to_s : new_value
+          @map[str_key] = new_value
         end
       end
     end

data/lib/contrast/configuration.rb

@@ -48,9 +48,7 @@ module Contrast
     # in an infinite loop on the to_sym method used later.
     def method_missing symbol, *args
       with_contrast_scope do
-        root.public_send(symbol, *args)
-      rescue NoMethodError => _e
-        super
+        root.public_send(symbol, *args) if root.cs__respond_to?(symbol)
       end
     end
 
@@ -101,8 +99,7 @@ module Contrast
       {}
     end
 
-    # We're updating properties loaded from the configuration
-    # files to match the new agreed upon standard configuration
+    # We're updating properties loaded from the configuration files to match the new agreed upon standard configuration
     # names, so that one file works for all agents
     def update_prop_keys config
       CONVERSION.each_pair do |old_method, new_method|
@@ -120,16 +117,7 @@ module Contrast
         # We changed the seconds values into ms values. Multiply them accordingly
         old_value = old_value.to_i * 1000 if new_method.end_with?(MILLISECOND_MARKER)
         new_value = config
-        end_idx = new_keys.length - 1
-        new_keys.each_with_index do |new_key, index|
-          if index == end_idx
-            new_value[new_key] = old_value if new_value[new_key].nil?
-          else
-            new_value = {} if new_value.nil?
-            new_value[new_key] = {} if new_value[new_key].nil?
-            new_value = new_value[new_key]
-          end
-        end
+        replace_props(new_keys, new_value, old_value)
       end
 
       config
@@ -237,5 +225,21 @@ module Contrast
         convert
       end
     end
+
+    def replace_props new_keys, new_value, old_value
+      idx = 0
+      end_idx = new_keys.length - 1
+      while idx < new_keys.length
+        new_key = new_keys[idx]
+        if idx == end_idx
+          new_value[new_key] = old_value if new_value[new_key].nil?
+        else
+          new_value = {} if new_value.nil?
+          new_value[new_key] = {} if new_value[new_key].nil?
+          new_value = new_value[new_key]
+        end
+        idx += 1
+      end
+    end
   end
 end

data/lib/contrast/extension/assess/array.rb

@@ -11,7 +11,7 @@ module Contrast
       # This is our patch of the Array class required to handle propagation
       # Disclaimer: there may be a better way, but we're in a 'get it work' state.
       # Hopefully, we'll be in a 'get it right' state soon.
-      class ArrayPropagator
+      class ArrayPropagator # rubocop:disable Style/StaticClass
         extend Contrast::Components::Scope::InstanceMethods
 
         ARRAY_JOIN_HASH = {
@@ -59,16 +59,6 @@ module Contrast
               ret
             end
           end
-
-          def instrument_array_track
-            @_instrument_array_track ||= begin
-              require 'cs__assess_array/cs__assess_array'
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading assess track patch', e)
-            false
-          end
         end
       end
     end

data/lib/contrast/extension/assess/eval_trigger.rb

@@ -34,26 +34,6 @@ module Contrast
                                                                               ret, source)
           end
 
-          def instrument_basic_object_track
-            @_instrument_basic_object_track ||= begin
-              require 'cs__assess_basic_object/cs__assess_basic_object'
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading basic object track patch', e)
-            false
-          end
-
-          def instrument_module_track
-            @_instrument_module_track ||= begin
-              require 'cs__assess_module/cs__assess_module'
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading module track patch', e)
-            false
-          end
-
           private
 
           def trigger_node clazz, method

data/lib/contrast/extension/assess/fiber.rb

@@ -20,7 +20,6 @@ module Contrast
         extend Contrast::Components::Logger::InstanceMethods
         extend Contrast::Components::Scope::InstanceMethods
 
-
         # we use funchook to patch rb_fiber_new the initialize method is not exposed by Ruby core
         FIBER_NEW_NODE_HASH = {
             'class_name' => 'Fiber',
@@ -86,16 +85,6 @@ module Contrast
           rescue Exception => e # rubocop:disable Lint/RescueException
             logger.error('Unable to propagate during Fiber.new', e)
           end
-
-          def instrument_fiber_track
-            @_instrument_fiber_variables ||= begin
-              require 'cs__assess_fiber_track/cs__assess_fiber_track' if Funchook.available?
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading fiber track patch', e)
-            false
-          end
         end
       end
     end

data/lib/contrast/extension/assess/hash.rb

@@ -25,16 +25,6 @@ module Contrast
             # result in a seg fault
             object
           end
-
-          def instrument_hash_track
-            @_instrument_hash_track ||= begin
-              require 'cs__assess_hash/cs__assess_hash'
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading hash track patch', e)
-            false
-          end
         end
       end
     end

data/lib/contrast/extension/assess/kernel.rb

@@ -14,6 +14,7 @@ module Contrast
       module KernelPropagator
         class << self
           extend Contrast::Components::Logger::InstanceMethods
+          include Contrast::Components::Logger::InstanceMethods
           include Contrast::Extension::Assess::ExecTrigger
 
           # We're 'tracking' sprintf now, meaning if anything is tracked on the way
@@ -65,16 +66,6 @@ module Contrast
             logger.error('Unable to track dataflow through sprintf', e)
           end
 
-          def instrument_kernel_track
-            @_instrument_fiber_variables ||= begin
-              require 'cs__assess_kernel/cs__assess_kernel'
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading kernel track patch', e)
-            false
-          end
-
           private
 
           def handle_sprintf_value value, result, parent_events

data/lib/contrast/extension/assess/marshal.rb

@@ -12,10 +12,12 @@ module Contrast
       # Hopefully, we'll be in a 'get it right' state soon.
       # This module is used for our Marshal.load patches
       class MarshalPropagator
-        extend Contrast::Components::Logger::InstanceMethods
         extend Contrast::Components::Scope::InstanceMethods
 
         class << self
+          extend Contrast::Components::Logger::InstanceMethods
+          include Contrast::Components::Logger::InstanceMethods
+
           def cs__load_protect arg
             return if in_contrast_scope?
 
@@ -44,16 +46,6 @@ module Contrast
             end
           end
 
-          def instrument_marshal_load
-            @_instrument_marshal_load ||= begin
-              require 'cs__assess_marshal_module/cs__assess_marshal_module'
-              true
-            end
-          rescue StandardError, LoadError => e
-            logger.error('Error loading marshal load patch', e)
-            false
-          end
-
           def trigger_node clazz, method
             triggers = Contrast::Agent::Assess::Policy::Policy.instance.triggers
             return unless triggers