Merge pull request #16966 from why-el/symbolize-path-params

Ensure named path params are symbols (Fixes #16958)

2 files changed, 9 insertions, 0 deletions

diff --git a/actionpack/lib/action_dispatch/routing/route_set.rb b/actionpack/lib/action_dispatch/routing/route_set.rb
index f51bee3b31..602d2b3944 100644
--- a/actionpack/lib/action_dispatch/routing/route_set.rb
+++ b/actionpack/lib/action_dispatch/routing/route_set.rb
@@ -287,6 +287,7 @@ module ActionDispatch
               }
             end
 
+            inner_options.symbolize_keys!
             result.merge!(inner_options)
           end
         end
diff --git a/actionpack/test/dispatch/routing/route_set_test.rb b/actionpack/test/dispatch/routing/route_set_test.rb
index c465d56bde..87d44064ea 100644
--- a/actionpack/test/dispatch/routing/route_set_test.rb
+++ b/actionpack/test/dispatch/routing/route_set_test.rb
@@ -80,6 +80,14 @@ module ActionDispatch
         assert_equal '/foo/1/bar/2', url_helpers.foo_bar_path(2, foo_id: 1)
       end
 
+      test "stringified controller and action keys are properly symbolized" do
+        draw do
+          root 'foo#bar'
+        end
+
+        assert_equal '/', url_helpers.root_path('controller' => 'foo', 'action' => 'bar')
+      end
+
       private
         def draw(&block)
           @set.draw(&block)