Merge pull request #1655 from presidentbeef/root_in_routes

Calls to `root` in routes might not be `root to: ...`
presidentbeef · Dec 15, 2021 · e05a6e8 · e05a6e8
2 parents 4585df4 + 743eebd
commit e05a6e8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/lib/brakeman/processors/lib/rails3_route_processor.rb b/lib/brakeman/processors/lib/rails3_route_processor.rb
@@ -78,6 +78,8 @@ def process_namespace exp
 
   #TODO: Need test for this
   def process_root exp
+    return exp unless hash? exp.first_arg
+
     if value = hash_access(exp.first_arg, :to)
       if string? value
         add_route_from_string value

diff --git a/test/apps/rails4/config/routes.rb b/test/apps/rails4/config/routes.rb
@@ -1,3 +1,5 @@
+something.root
+
 Rails4::Application.routes.draw do
   resources not_a_symbol, :controller => :whatever