From d859a5d5c655e06895f80c209a98c135dc5ece74 Mon Sep 17 00:00:00 2001 From: Justin Collins Date: Wed, 5 Sep 2012 18:22:38 -0700 Subject: [PATCH] Fix rescanning when only controller action changes Basically, because the tracker caches the template names and the rescanner was deleting the templates without clearing the cache. Should use Tracker#reset_template instead --- lib/brakeman/rescanner.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/brakeman/rescanner.rb b/lib/brakeman/rescanner.rb index da3f21d7..b46da881 100644 --- a/lib/brakeman/rescanner.rb +++ b/lib/brakeman/rescanner.rb @@ -117,7 +117,7 @@ class Brakeman::Rescanner < Brakeman::Scanner if controller[:file] == path tracker.templates.keys.each do |template_name| if template_name.to_s.match /(.+)\.#{name}#/ - tracker.templates.delete template_name + tracker.reset_template template_name end end @@ -222,7 +222,7 @@ class Brakeman::Rescanner < Brakeman::Scanner #Remove templates rendered from this controller tracker.templates.keys.each do |template_name| if template_name.to_s.match template_matcher - tracker.templates.delete template_name + tracker.reset_template template_name end end -- GitLab