kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Restrict matching with word boundary or end of string

Browse source
This commit is contained in:
Petri Avikainen 2019-02-05 14:13:38 +02:00
parent c94754e2f0
commit 1315582b37
No known key found for this signature in database
GPG key ID: D2315E803563CED4
2 changed files with 12 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
actionpack/lib/action_dispatch/journey/path/pattern.rb
View file

@ -119,7 +119,7 @@ module ActionDispatch
class UnanchoredRegexp < AnchoredRegexp # :nodoc:
def accept(node)
%r{\A#{visit node}\b}
%r{\A#{visit node}(?:\b|\Z)}
end
end

22
actionpack/test/journey/path/pattern_test.rb
View file

@ -34,17 +34,17 @@ module ActionDispatch
end
{
"/:controller(/:action)" => %r{\A/(#{x})(?:/([^/.?]+))?\b},
"/:controller/foo" => %r{\A/(#{x})/foo\b},
"/:controller/:action" => %r{\A/(#{x})/([^/.?]+)\b},
"/:controller" => %r{\A/(#{x})\b},
"/:controller(/:action(/:id))" => %r{\A/(#{x})(?:/([^/.?]+)(?:/([^/.?]+))?)?\b},
"/:controller/:action.xml" => %r{\A/(#{x})/([^/.?]+)\.xml\b},
"/:controller.:format" => %r{\A/(#{x})\.([^/.?]+)\b},
"/:controller(.:format)" => %r{\A/(#{x})(?:\.([^/.?]+))?\b},
"/:controller/*foo" => %r{\A/(#{x})/(.+)\b},
"/:controller/*foo/bar" => %r{\A/(#{x})/(.+)/bar\b},
"/:foo|*bar" => %r{\A/(?:([^/.?]+)|(.+))\b},
"/:controller(/:action)" => %r{\A/(#{x})(?:/([^/.?]+))?(?:\b|\Z)},
"/:controller/foo" => %r{\A/(#{x})/foo(?:\b|\Z)},
"/:controller/:action" => %r{\A/(#{x})/([^/.?]+)(?:\b|\Z)},
"/:controller" => %r{\A/(#{x})(?:\b|\Z)},
"/:controller(/:action(/:id))" => %r{\A/(#{x})(?:/([^/.?]+)(?:/([^/.?]+))?)?(?:\b|\Z)},
"/:controller/:action.xml" => %r{\A/(#{x})/([^/.?]+)\.xml(?:\b|\Z)},
"/:controller.:format" => %r{\A/(#{x})\.([^/.?]+)(?:\b|\Z)},
"/:controller(.:format)" => %r{\A/(#{x})(?:\.([^/.?]+))?(?:\b|\Z)},
"/:controller/*foo" => %r{\A/(#{x})/(.+)(?:\b|\Z)},
"/:controller/*foo/bar" => %r{\A/(#{x})/(.+)/bar(?:\b|\Z)},
"/:foo|*bar" => %r{\A/(?:([^/.?]+)|(.+))(?:\b|\Z)},
}.each do |path, expected|
define_method(:"test_to_non_anchored_regexp_#{Regexp.escape(path)}") do
path = Pattern.build(