diff options
author | Gareth Rees <gareth@mysociety.org> | 2014-10-01 13:00:01 +0100 |
---|---|---|
committer | Louise Crow <louise.crow@gmail.com> | 2014-12-22 16:33:13 +0000 |
commit | b6cd9b05ae27370088c04973413730040f4ff08c (patch) | |
tree | e533edd7b65ebb89a2e04eb4feb0bda2bf73c1dc /app/controllers/user_controller.rb | |
parent | 5fc4e4992069088684e40cabdf7128cf6f9ad67d (diff) |
Fix unvalidated redirects
Diffstat (limited to 'app/controllers/user_controller.rb')
-rw-r--r-- | app/controllers/user_controller.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb index 70036341c..c4ce28934 100644 --- a/app/controllers/user_controller.rb +++ b/app/controllers/user_controller.rb @@ -256,7 +256,7 @@ class UserController < ApplicationController def signout self._do_signout if params[:r] - redirect_to params[:r] + redirect_to URI.parse(params[:r]).path else redirect_to :controller => "general", :action => "frontpage" end @@ -596,7 +596,7 @@ class UserController < ApplicationController end @user.receive_email_alerts = params[:receive_email_alerts] @user.save! - redirect_to params[:came_from] + redirect_to URI.parse(params[:came_from]).path end private |