This project is archived and is in readonly mode.
Rack::Mount blows up if passed invalid constraints
#4989
If you use scope to specify some constraints that aren't used within a route inside that scope then Rack::Mount raises an exception. The attached patch removes any constraints that are not in the list segment keys or is not a public instance method on the request class.
-
Andrew WhiteAn example routes.rb file that shows this error is:
scope :constraints => { :id => /\d+/ } do resources :products resources :images end -
Andrew White
Updated patch to use only instance variables and pass in RouteSet as first parameter.
-
Repository- State changed from open to resolved
(from [19ccd4628c1ab4aebc00dc8d480d6cbf1688a312]) Remove invalid conditions from route [#4989 Rack::Mount blows up if passed invalid constraints state:resolved]
Signed-off-by: José Valim jose.valim@gmail.com
http://github.com/rails/rails/commit/19ccd4628c1ab4aebc00dc8d480d6c... -
