aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/routing/redirection.rb
diff options
context:
space:
mode:
authorGrey Baker <greysteil@gmail.com>2016-07-13 18:44:04 +0100
committerGrey Baker <greysteil@gmail.com>2016-07-14 16:50:49 +0100
commit9f38a3fb0c9c71102da283b014503ccad92da581 (patch)
tree0bfaa2eeb07129d446ff8441ab9d4354806e7391 /actionpack/lib/action_dispatch/routing/redirection.rb
parentea31bdd7c8b1221d97de2392ac22d9c5fb8657d5 (diff)
downloadrails-9f38a3fb0c9c71102da283b014503ccad92da581.tar.gz
rails-9f38a3fb0c9c71102da283b014503ccad92da581.tar.bz2
rails-9f38a3fb0c9c71102da283b014503ccad92da581.zip
Check `request.path_parameters` encoding at the point they're set
Check for any non-UTF8 characters in path parameters at the point they're set in `env`. Previously they were checked for when used to get a controller class, but this meant routes that went directly to a Rack app, or skipped controller instantiation for some other reason, had to defend against non-UTF8 characters themselves.
Diffstat (limited to 'actionpack/lib/action_dispatch/routing/redirection.rb')
-rw-r--r--actionpack/lib/action_dispatch/routing/redirection.rb1
1 files changed, 0 insertions, 1 deletions
diff --git a/actionpack/lib/action_dispatch/routing/redirection.rb b/actionpack/lib/action_dispatch/routing/redirection.rb
index d6987f4d09..3265caa00b 100644
--- a/actionpack/lib/action_dispatch/routing/redirection.rb
+++ b/actionpack/lib/action_dispatch/routing/redirection.rb
@@ -22,7 +22,6 @@ module ActionDispatch
end
def serve(req)
- req.check_path_parameters!
uri = URI.parse(path(req.path_parameters, req))
unless uri.host