diff options
author | Gareth Rees <gareth@mysociety.org> | 2014-03-27 16:53:09 +0000 |
---|---|---|
committer | Gareth Rees <gareth@mysociety.org> | 2014-03-31 16:28:57 +0100 |
commit | ee2d0f30b7699248c2ace02c12ce7223102b6077 (patch) | |
tree | 882ca5b0622fa48fedd385ba53fad35a935dd871 /script/rails-deploy-before-down | |
parent | 0adf9399cbef42054809479c8f1b64dad7bbf8ca (diff) |
URL Encode the path parameter for render_exception
If a request is made and path is something like /%d3 we rescue this with
a custom 404 template.
This gets unescaped as {"path"=>"\323"}.
In the case of a RouteNotFound, ApplicationController#render_exception
renders the general/exception_caught template in to the default layout,
which renders the general/_locale_switcher partial.
This partial calls url_for – sending the full params hash as the
argument – so that a user may return to the existing page in their
chosen locale.
The problem is that url_for tries to construct the url with the hash
{:action=>"not_found", :controller=>"general",
:path=>"\323"}.
ApplicationController#sanitize_params re-encodes the path parameter so
that it can be passed through to url_for without trouble.
Diffstat (limited to 'script/rails-deploy-before-down')
0 files changed, 0 insertions, 0 deletions