From 10e0ebbd9dfbd9436226330e0962a0d28dbf2237 Mon Sep 17 00:00:00 2001 From: Dave Riess Date: Sat, 27 Dec 2014 12:20:38 -0800 Subject: [PATCH] replace request.path + '?' + request.query_string with request.original_fullpath in order to avoid appending '?' to redirects that have no query string --- lib/two_factor_authentication/controllers/helpers.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/two_factor_authentication/controllers/helpers.rb b/lib/two_factor_authentication/controllers/helpers.rb index a622ddea..505ca019 100644 --- a/lib/two_factor_authentication/controllers/helpers.rb +++ b/lib/two_factor_authentication/controllers/helpers.rb @@ -21,7 +21,7 @@ def handle_two_factor_authentication def handle_failed_second_factor(scope) if request.format.present? and request.format.html? - session["#{scope}_return_to"] = "#{request.path}?#{request.query_string}" if request.get? + session["#{scope}_return_to"] = request.original_fullpath if request.get? redirect_to two_factor_authentication_path_for(scope) else render nothing: true, status: :unauthorized