Sha256: 17956e920a4d25c2f7e6df9bd8cfc2eafd777ff78259b0c05fa9919f803f3641
Contents?: true
Size: 1.26 KB
Versions: 13
Compression:
Stored size: 1.26 KB
Contents
require 'brakeman/checks/base_check' # Checks for unscoped calls to models' #find and #find_by_id methods. class Brakeman::CheckUnscopedFind < Brakeman::BaseCheck Brakeman::Checks.add_optional self @description = "Check for unscoped ActiveRecord queries" def run_check Brakeman.debug("Finding instances of #find on models with associations") associated_model_names = active_record_models.keys.select do |name| active_record_models[name].associations[:belongs_to] end calls = tracker.find_call :method => [:find, :find_by_id, :find_by_id!], :targets => associated_model_names calls.each do |call| process_result call end end def process_result result return if duplicate? result or result[:call].original_line # Not interested unless argument is user controlled. inputs = result[:call].args.map { |arg| include_user_input?(arg) } return unless input = inputs.compact.first add_result result warn :result => result, :warning_type => "Unscoped Find", :warning_code => :unscoped_find, :message => "Unscoped call to #{result[:target]}##{result[:method]}", :code => result[:call], :confidence => :weak, :user_input => input end end
Version data entries
13 entries across 13 versions & 3 rubygems