contrast-agent 6.7.0 → 6.9.0

data/lib/contrast/agent/protect/rule/path_traversal/path_traversal_semantic_security_bypass.rb

@@ -0,0 +1,114 @@
+# Copyright (c) 2022 Contrast Security, Inc. See https://www.contrastsecurity.com/enduser-terms-0317a for more details.
+# frozen_string_literal: true
+
+require 'contrast/agent/protect/rule/base_service'
+require 'contrast/agent/reporting/details/path_traversal_semantic_analysis_details'
+require 'contrast/agent/request_context'
+require 'contrast/utils/string_utils'
+require 'contrast/agent_lib/api/path_semantic_file_security_bypass'
+require 'contrast/agent_lib/interface'
+
+module Contrast
+  module Agent
+    module Protect
+      module Rule
+        # The Ruby implementation of the Protect Path Traversal Semantic
+        # Bypass sub-rule. This rule should report the attack result
+        class PathTraversalSemanticBypass < Contrast::Agent::Protect::Rule::BaseService
+          NAME = 'path-traversal-semantic-file-security-bypass'
+          # There paths here will eventually be moved
+
+          def rule_name
+            NAME
+          end
+
+          def infilter context, method, path
+            return unless rule_violated?(path)
+
+            result = find_vulnerability(context, path)
+
+            return unless result
+
+            append_to_activity(context, result)
+            return unless blocked?
+
+            result_rule_name = Contrast::Utils::StringUtils.transform_string(result.rule_id)
+            cef_logging(result, :successful_attack, value: path)
+            exception_messasge = "#{ result_rule_name } rule triggered. Call to File.#{ method } blocked."
+            raise(Contrast::SecurityException.new(self, exception_messasge))
+          end
+
+          protected
+
+          # Check if semantic file security bypass is detected
+          #
+          # @param file_path [String] command to check.
+          # @param is_custom_code[String] whether the file is being accessed by custom (user) code,
+          #     rather than framework code
+          # @return result[Integer, nil] returns:
+          #     1 => security bypass is detected.
+          #     0 => no security bypass is detected.
+          def file_security_bypassed? file_path, is_custom_code = nil
+            return false unless (agent_lib = Contrast::AGENT_LIB) || file_path
+
+            custom_call = is_custom_code.nil? ? 0 : 1
+
+            agent_lib.check_path_semantic_security_bypass(file_path, custom_call) == 1
+          end
+
+          def rule_violated? path
+            is_custom_code = custom_code_accessing_system_file?(path)
+            is_custom_code || file_security_bypassed?(path, is_custom_code)
+          end
+
+          def find_vulnerability context, path
+            build_attack_with_match(context, nil, nil, path)
+          end
+
+          def build_sample context, _input_analysis_result, path, **_kwargs
+            sample = build_base_sample(context, nil)
+            sample.details = Contrast::Agent::Reporting::Details::PathTraversalSemanticAnalysisDetails.new
+            path = Contrast::Utils::StringUtils.protobuf_safe_string(path)
+            sample.details.path = path
+
+            is_custom_code = custom_code_accessing_system_file?(path)
+            # This should catch all the types of security breaches in that sub-rule scope
+            # but apparently it's not, because some of the system files is being skipped and not detected,
+            # but for our previous logic - it was expected for certain files to be detected and blocked
+            security_bypassed = file_security_bypassed?(path, is_custom_code)
+
+            # if agent lib sub-rule returns true and is custom code -> assign and report
+            if security_bypassed
+              if is_custom_code
+                sample.details.findings << :CUSTOM_CODE_ACCESSING_SYSTEM_FILES
+                sample.details.findings << :COMMON_FILE_EXPLOITS if common_file_exploits_enabled?
+              end
+              return sample
+            end
+
+            if is_custom_code
+              sample.details.findings << :CUSTOM_CODE_ACCESSING_SYSTEM_FILES
+              return sample
+            end
+
+            nil
+          end
+
+          def custom_code_accessing_system_file? input
+            system_file?(input) && Contrast::Utils::StackTraceUtils.custom_code_context?
+          end
+
+          def system_file? path
+            return false unless path
+
+            Contrast::Agent::Protect::Rule::PathTraversal::SYSTEM_PATHS.any? { |sys_path| sys_path.include?(path) }
+          end
+
+          def common_file_exploits_enabled?
+            false
+          end
+        end
+      end
+    end
+  end
+end

data/lib/contrast/agent/protect/rule/path_traversal.rb

@@ -2,9 +2,13 @@
 # frozen_string_literal: true
 
 require 'contrast/agent/protect/rule/base_service'
+require 'contrast/agent/protect/rule/path_traversal/path_traversal_semantic_security_bypass'
+require 'contrast/agent/reporting/input_analysis/input_type'
+require 'contrast/agent/reporting/input_analysis/score_level'
 require 'contrast/utils/stack_trace_utils'
 require 'contrast/agent/reporting/details/path_traversal_details'
 require 'contrast/agent/reporting/details/path_traversal_semantic_analysis_details'
+require 'contrast/utils/string_utils'
 
 module Contrast
   module Agent
@@ -13,7 +17,13 @@ module Contrast
         # This class handles our implementation of the Path Traversal
         # Protect rule.
         class PathTraversal < Contrast::Agent::Protect::Rule::BaseService
+          include Contrast::Agent::Reporting::InputType
+
           NAME = 'path-traversal'
+          APPLICABLE_USER_INPUTS = [
+            BODY, COOKIE_NAME, COOKIE_VALUE, HEADER, PARAMETER_VALUE, PARAMETER_NAME,
+            JSON_VALUE, MULTIPART_VALUE, MULTIPART_FIELD_NAME, XML_VALUE, DWR_VALUE, URI
+          ].cs__freeze
           SYSTEM_PATHS = %w[
             /proc/self
             etc/passwd
@@ -27,20 +37,21 @@ module Contrast
             /windows/repair/
           ].cs__freeze
 
-          class << self
-            # @param attack_sample [Contrast::Api::Dtm::RaspRuleSample]
-            # @return [Hash] the details for this specific rule
-            def extract_details attack_sample
-              {
-                  path: attack_sample.path_traversal.path
-              }
-            end
-          end
-
           def rule_name
             NAME
           end
 
+          # Array of sub_rules
+          #
+          # @return [Array]
+          def sub_rules
+            @_sub_rules ||= [Contrast::Agent::Protect::Rule::PathTraversalSemanticBypass.new].cs__freeze
+          end
+
+          def applicable_user_inputs
+            APPLICABLE_USER_INPUTS
+          end
+
           def infilter context, method, path
             return unless infilter?(context)
 
@@ -50,9 +61,10 @@ module Contrast
             append_to_activity(context, result)
             return unless blocked?
 
-            cef_logging(result, :successful_attack)
-            raise(Contrast::SecurityException.new(self,
-                                                  "Path Traversal rule triggered. Call to File.#{ method } blocked."))
+            result_rule_name = Contrast::Utils::StringUtils.transform_string(result.rule_id)
+            cef_logging(result, :successful_attack, value: path)
+            exception_messasge = "#{ result_rule_name } rule triggered. Call to File.#{ method } blocked."
+            raise(Contrast::SecurityException.new(self, exception_messasge))
           end
 
           protected
@@ -73,6 +85,21 @@ module Contrast
             sample
           end
 
+          # @param context [Contrast::Agent::RequestContext]
+          def infilter? context
+            return false unless enabled?
+            return false unless context&.agent_input_analysis&.results&.any? do |result|
+              # When a file is being accessed, the agent should see if any of its worth-watching inputs appear in
+              # the file path. If so, the input is considered a confirmed attack and should be reported or blocked.
+              # If the score level is ignore we don't need to report it.
+              result.rule_id == rule_name && result.score_level != Contrast::Agent::Reporting::ScoreLevel::IGNORE
+            end
+
+            return false if protect_excluded_by_code?
+
+            true
+          end
+
           private
 
           # Build a subclass of the RaspRuleSample if the sample matches

data/lib/contrast/agent/protect/rule/sql_sample_builder.rb

@@ -15,11 +15,11 @@ module Contrast
           # by TeamServer
           #
           # @param context [Contrast::Agent::RequestContext] the context for the current request
-          # @param input_analysis_result [Contrast::Api::Dtm::AttackResult, nil] previous attack result for this rule,
+          # @param input_analysis_result [Contrast::Agent::Reporting, nil] previous attack result for this rule,
           #   if one exists, in the case of multiple inputs being found to violate the protection criteria
           # @candidate_string [String] the value of the input which may be an attack
           # @kwargs [Hash] key - value pairs of context individual rules need to build out details
-          #   to send to the Service to tell the story of the attack
+          #   to send to TeamServer to tell the story of the attack
           # @return [Contrast::Agent::Reporting::RaspRuleSample] the sample from this attack
           module SqliSample
             def build_sample context, input_analysis_result, candidate_string, **kwargs
@@ -38,11 +38,11 @@ module Contrast
           # by TeamServer
           #
           # @param context [Contrast::Agent::RequestContext] the context for the current request
-          # @param input_analysis_result [Contrast::Api::Dtm::AttackResult, nil] previous attack result for this rule,
+          # @param input_analysis_result [Contrast::Agent::Reporting, nil] previous attack result for this rule,
           #   if one exists, in the case of multiple inputs being found to violate the protection criteria
           # @candidate_string [String] the value of the input which may be an attack
           # @kwargs [Hash] key - value pairs of context individual rules need to build out details
-          #   to send to the Service to tell the story of the attack
+          #   to send to TeamServer to tell the story of the attack
           # @return [Contrast::Agent::Reporting::RaspRuleSample] the sample from this attack
           module NoSqliSample
             def build_sample context, input_analysis_result, candidate_string, **kwargs
@@ -63,24 +63,21 @@ module Contrast
             # Set up an attack result and assigns Database scanner for the No-SQL and SQLI injection detection rules
             #
             # @param context [Contrast::Agent::RequestContext] the context for the current request
-            # @param input_analysis_result [Contrast::Api::Dtm::AttackResult, nil] previous attack result for this rule,
+            # @param input_analysis_result [Contrast::Agent::Reporting, nil] previous attack result for this rule,
             #   if one exists, in the case of multiple inputs being found to violate the protection criteria
-            # @param result [Contrast::Api::Dtm::AttackResult, nil] previous attack result for this rule, if one exists,
+            # @param result [Contrast::Agent::Reporting, nil] previous attack result for this rule, if one exists,
             #   in the case of multiple inputs being found to violate the protection criteria
             # @param query_string [String] the value of the input which may be an attack
             # @param kwargs [Hash] key - value pairs of context individual rules need to build out details to send
-            #   to the Service to tell the story of the attack
-            # @return [Contrast::Api::Dtm::AttackResult] the result from this attack
+            #   to TeamServer to tell the story of the attack
+            # @return [Contrast::Agent::Reporting] the result from this attack
             def build_attack_with_match context, input_analysis_result, result, query_string, **kwargs
-              if mode == Contrast::Api::Settings::ProtectionRule::Mode::NO_ACTION ||
-                    mode == Contrast::Api::Settings::ProtectionRule::Mode::PERMIT
-
-                return result
-              end
+              return result if mode == :NO_ACTION || mode == :PERMIT
 
               attack_string = input_analysis_result.value
               regexp = Regexp.new(Regexp.escape(attack_string), Regexp::IGNORECASE)
-
+              # extract struct result from kwargs
+              agent_lib_struct_result = kwargs[:result_struct]
               return unless query_string.match?(regexp)
 
               database = kwargs[:database]
@@ -94,7 +91,14 @@ module Contrast
                 next unless last_boundary && boundary
 
                 result ||= build_attack_result(context)
-                record_match(idx, length, boundary, last_boundary, kwargs)
+
+                # if the struct actually has the needed data in it - use it
+                if agent_lib_struct_result.cs__is_a?(Hash)
+                  record_agent_lib_match(agent_lib_struct_result, length, kwargs)
+                else
+                  record_match(idx, length, boundary, last_boundary, kwargs)
+                end
+
                 append_match(context, input_analysis_result, result, query_string, **kwargs)
               end
 
@@ -124,6 +128,20 @@ module Contrast
               kwargs[:input_boundary_idx] = last_boundary
             end
 
+            # all the agent_lib checks methods needed
+
+            # @param struct[ResultingStruct] The struct including all the data from the agent_lib scan
+            def record_agent_lib_match struct, length, kwargs
+              kwargs[:start_idx] = struct[:start_index]
+              kwargs[:end_idx] = if (struct[:end_index]).zero?
+                                   struct[:start_index] + length
+                                 else
+                                   struct[:end_index]
+                                 end
+              kwargs[:boundary_overrun_idx] = struct[:boundary_overrun_index]
+              kwargs[:input_boundary_idx] = struct[:input_boundary_index]
+            end
+
             def append_match context, input_analysis_result, result, query_string, **kwargs
               input_analysis_result.attack_count = input_analysis_result.attack_count + 1
               update_successful_attack_response(context, input_analysis_result, result, query_string)

data/lib/contrast/agent/protect/rule/sqli/sqli_base_rule.rb

@@ -19,20 +19,6 @@ module Contrast
             XML_VALUE, DWR_VALUE
           ].cs__freeze
 
-          class << self
-            # @param attack_sample [Contrast::Api::Dtm::RaspRuleSample]
-            # @return [Hash] the details for this specific rule
-            def extract_details attack_sample
-              {
-                  start: attack_sample.sqli.start_idx,
-                  end: attack_sample.sqli.end_idx,
-                  boundaryOverrunIndex: attack_sample.sqli.boundary_overrun_idx,
-                  inputBoundaryIndex: attack_sample.sqli.input_boundary_idx,
-                  query: attack_sample.sqli.query
-              }
-            end
-          end
-
           def infilter context, database, query_string
             return unless infilter?(context)
 

data/lib/contrast/agent/protect/rule/sqli/sqli_input_classification.rb

@@ -5,10 +5,8 @@ require 'contrast/utils/object_share'
 require 'contrast/agent/reporting/input_analysis/input_type'
 require 'contrast/agent/protect/rule/sqli'
 require 'contrast/agent/reporting/input_analysis/score_level'
-require 'contrast/agent/protect/rule/sqli/sqli_worth_watching'
 require 'contrast/agent/protect/input_analyzer/input_analyzer'
-require 'contrast/utils/input_classification'
-require 'contrast/components/logger'
+require 'contrast/utils/input_classification_base'
 
 module Contrast
   module Agent
@@ -18,68 +16,10 @@ module Contrast
         # as a result input would be marked as WORTHWATCHING or IGNORE,
         # to be analyzed at the sink level.
         module SqliInputClassification
+          WORTHWATCHING_MATCH = 'sqli-worth-watching-v2'.cs__freeze
           class << self
             include InputClassificationBase
-            include Contrast::Agent::Protect::Rule::SqliWorthWatching
             include Contrast::Components::Logger::InstanceMethods
-
-            WORTHWATCHING_MATCH = 'sqli-worth-watching-v2'
-            SQLI_KEYS_NEEDED = [
-              COOKIE_VALUE, PARAMETER_VALUE, JSON_VALUE, MULTIPART_VALUE, XML_VALUE, DWR_VALUE
-            ].cs__freeze
-
-            # Input Classification stage is done to determine if an user input is
-            # WORTHWATCHING or to be ignored.
-            #
-            # @param input_type [Contrast::Agent::Reporting::InputType] The type of the user input.
-            # @param value [String, Array<String>] the value of the input.
-            # @param input_analysis [Contrast::Agent::Reporting::InputAnalysis] Holds all the results from the
-            #                                                       agent analysis from the current
-            #                                                       Request.
-            # @return ia [Contrast::Agent::Reporting::InputAnalysis] with updated results.
-            def classify input_type, value, input_analysis
-              return unless Contrast::Agent::Protect::Rule::Sqli::APPLICABLE_USER_INPUTS.include?(input_type)
-              return unless super
-
-              rule_id = Contrast::Agent::Protect::Rule::Sqli::NAME
-
-              # double check the input to avoid calling match? on array
-              Array(value).each do |val|
-                Array(val).each do |v|
-                  input_analysis.results << sqli_create_new_input_result(input_analysis.request, rule_id, input_type, v)
-                end
-              end
-
-              input_analysis
-            rescue StandardError => e
-              logger.debug('An Error was recorded in the input classification of the sqli.')
-              logger.debug(e)
-            end
-
-            private
-
-            # This methods checks if input is tagged WORTHWATCHING or IGNORE matches value with it's
-            # key if needed and Creates new isntance of InputAnalysisResult.
-            #
-            # @param request [Contrast::Agent::Request] the current request context.
-            # @param rule_id [String] The name of the Protect Rule.
-            # @param input_type [Contrast::Agent::Reporting::InputType] The type of the user input.
-            # @param value [String, Array<String>] the value of the input.
-            #
-            # @return res [Contrast::Agent::Reporting::InputAnalysisResult]
-            def sqli_create_new_input_result request, rule_id, input_type, value
-              ia_result = new_ia_result(rule_id, input_type, request.path, value)
-              if sqli_worth_watching?(value)
-                ia_result.ids << WORTHWATCHING_MATCH
-                ia_result.score_level = WORTHWATCHING
-                ia_result
-              else
-                ia_result.score_level = IGNORE
-              end
-
-              add_needed_key(request, ia_result, input_type, value) if SQLI_KEYS_NEEDED.include?(input_type)
-              ia_result
-            end
           end
         end
       end

data/lib/contrast/agent/protect/rule/sqli.rb

@@ -26,8 +26,6 @@ module Contrast
 
           NAME = 'sql-injection'
 
-          SUB_RULES = [Contrast::Agent::Protect::Rule::SqliDangerousFunctions.new].cs__freeze
-
           def rule_name
             NAME
           end
@@ -36,8 +34,81 @@ module Contrast
             BLOCK_MESSAGE
           end
 
+          # Array of sub_rules
+          #
+          # @return [Array]
           def sub_rules
-            SUB_RULES
+            @_sub_rules ||= [Contrast::Agent::Protect::Rule::SqliDangerousFunctions.new].cs__freeze
+          end
+
+          def applicable_user_inputs
+            APPLICABLE_USER_INPUTS
+          end
+
+          def infilter context, database, query_string
+            return unless infilter?(context)
+
+            result = find_attacker(context, query_string, database: database)
+            return unless result
+
+            append_to_activity(context, result)
+
+            cef_logging(result, :successful_attack, value: query_string)
+            raise(Contrast::SecurityException.new(self, BLOCK_MESSAGE)) if blocked?
+          end
+
+          # Allows for the InputAnalysis from Agent Library to be extracted early
+          # @param context [Contrast::Agent::RequestContext]
+          # @param potential_attack_string [String, nil]
+          # @param ia_results [Array<Contrast::Agent::Reporting::InputAnalysis>]
+          # @param **kwargs
+          # @return [Contrast::Agent::Reporting, nil]
+          def find_attacker_with_results context, potential_attack_string, ia_results, **kwargs
+            logger.trace('Checking vectors for attacks', rule: rule_name, input: potential_attack_string)
+
+            result = nil
+            ia_results.each do |ia_result|
+              if potential_attack_string
+                idx = potential_attack_string.index(ia_result.value)
+                next unless idx
+
+                database_type = kwargs[:database].to_sym
+                input_length = ia_result.value.length
+                lib_result = check_sql_input_with_agent(potential_attack_string, database_type, idx, input_length)
+
+                kwargs[:result_struct] = lib_result
+                result = build_attack_with_match(context, ia_result, result, potential_attack_string, **kwargs)
+              else
+                result = build_attack_without_match(context, ia_result, result, **kwargs)
+              end
+            end
+            result
+          end
+
+          # We'll need a second place, where we need to check the token boundaries if are being crossed and
+          # worth-watching.
+          #
+          # @param sql[String] SQL coming from parameter
+          # @param database[String] Type of database
+          # @param input_index[String] index in the sqlQuery string where user input was found
+          # @param input_length[Number] length of the input value
+          # @return [Hash, Boolean]
+          def check_sql_input_with_agent sql, database, input_index, input_length
+            return false unless (agent_lib = Contrast::AGENT_LIB) && sql && database
+
+            agent_lib.check_sql_query(input_index, input_length, database, sql)
+          end
+
+          # @param context [Contrast::Agent::RequestContext]
+          def infilter? context
+            return false unless enabled?
+            return false unless context&.agent_input_analysis&.results&.any? do |result|
+              result.rule_id == rule_name
+            end
+
+            return false if protect_excluded_by_code?
+
+            true
           end
         end
       end

data/lib/contrast/agent/protect/rule/unsafe_file_upload/unsafe_file_upload_input_classification.rb

@@ -3,9 +3,8 @@
 
 require 'contrast/utils/object_share'
 require 'contrast/agent/protect/rule/unsafe_file_upload'
-require 'contrast/agent/protect/rule/unsafe_file_upload/unsafe_file_upload_matcher'
 require 'contrast/agent/protect/input_analyzer/input_analyzer'
-require 'contrast/utils/input_classification'
+require 'contrast/utils/input_classification_base'
 
 module Contrast
   module Agent
@@ -14,67 +13,44 @@ module Contrast
         # This module will do the Input Classification stage of Unsafe File Upload
         # rule. As a result input would be marked as DEFINITEATTACK or IGNORE.
         module UnsafeFileUploadInputClassification
-          # UNSAFE_UPLOAD_MATCH = 'unsafe-file-upload-input-tracing-v1'
-          #
-          # class << self
-          #   include InputClassificationBase
-          #   include Contrast::Agent::Protect::Rule::UnsafeFileUploadMatcher
-          #
-          #   # Input Classification stage is done to determine if an user input is
-          #   # DEFINITEATTACK or to be ignored.
-          #   #
-          #   # @param input_type [Contrast::Agent::Reporting::InputType] The type of the user input.
-          #   # @param value [String, Array<String>] the value of the input.
-          #   # @param input_analysis [Contrast::Agent::Reporting::InputAnalysis] Holds all the results from the
-          #   #                                                       agent analysis from the current
-          #   #                                                       Request.
-          #   # @return ia [Contrast::Agent::Reporting::InputAnalysis] with updated results.
-          #   def classify input_type, value, input_analysis
-          #     unless Contrast::Agent::Protect::Rule::UnsafeFileUpload::APPLICABLE_USER_INPUTS.include?(input_type)
-          #       return
-          #     end
-          #     return unless input_analysis.request
-          #
-          #     rule_id = Contrast::Agent::Protect::Rule::UnsafeFileUpload::NAME
-          #     results = []
-          #
-          #     Array(value).each do |val|
-          #       Array(val).each do |v|
-          #         results << create_new_input_result(input_analysis.request, rule_id, input_type, v)
-          #       end
-          #     end
-          #
-          #     input_analysis.results = results
-          #     input_analysis
-          #   end
-          #
-          #   private
-          #
-          #   # This methods checks if input is tagged DEFINITEATTACK or IGNORE matches value with it's
-          #   # key if needed and Creates new isntance of InputAnalysisResult.
-          #   #
-          #   # @param request [Contrast::Agent::Request] the current request context.
-          #   # @param rule_id [String] The name of the Protect Rule.
-          #   # @param input_type [Contrast::Agent::Reporting::InputType] The type of the user input.
-          #   # @param value [String, Array<String>] the value of the input.
-          #   #
-          #   # @return res [Contrast::Agent::Reporting::InputAnalysisResult]
-          #   def create_new_input_result request, rule_id, input_type, value
-          #     ia_result = new_ia_result rule_id, input_type, request.path, value
-          #     if unsafe_match? value
-          #       ia_result.score_level = DEFINITEATTACK
-          #       ia_result.ids << UNSAFE_UPLOAD_MATCH
-          #     else
-          #       ia_result.score_level = IGNORE
-          #     end
-          #     ia_result.key = if input_type == MULTIPART_FIELD_NAME
-          #                       Contrast::Agent::Protect::InputAnalyzer::DISPOSITION_FILENAME
-          #                     else
-          #                       Contrast::Agent::Protect::InputAnalyzer::DISPOSITION_NAME
-          #                     end
-          #     ia_result
-          #   end
-          # end
+          UNSAFE_UPLOAD_MATCH = 'unsafe-file-upload-input-tracing-v1'
+
+          class << self
+            include InputClassificationBase
+
+            private
+
+            # This methods checks if input is tagged DEFINITEATTACK or IGNORE matches value with it's
+            # key if needed and Creates new isntance of InputAnalysisResult.
+            #
+            # @param request [Contrast::Agent::Request] the current request context.
+            # @param rule_id [String] The name of the Protect Rule.
+            # @param input_type [Contrast::Agent::Reporting::InputType] The type of the user input.
+            # @param value [String, Array<String>] the value of the input.
+            #
+            # @return res [Contrast::Agent::Reporting::InputAnalysisResult]
+            def create_new_input_result request, rule_id, input_type, value
+              return unless Contrast::AGENT_LIB
+              return unless (input_eval = Contrast::AGENT_LIB.eval_input(value,
+                                                                         Contrast::AGENT_LIB.input_set[:MULTIPART_NAME],
+                                                                         Contrast::AGENT_LIB.rule_set[rule_id],
+                                                                         Contrast::AGENT_LIB.eval_option[:NONE]))
+
+              ia_result = new_ia_result(rule_id, input_type, request.path, value)
+              if input_eval.score >= THRESHOLD
+                ia_result.score_level = DEFINITEATTACK
+                ia_result.ids << UNSAFE_UPLOAD_MATCH
+              else
+                ia_result.score_level = IGNORE
+              end
+              ia_result.key = if input_type == MULTIPART_FIELD_NAME
+                                Contrast::Agent::Protect::InputAnalyzer::DISPOSITION_FILENAME
+                              else
+                                Contrast::Agent::Protect::InputAnalyzer::DISPOSITION_NAME
+                              end
+              ia_result
+            end
+          end
         end
       end
     end