diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2015-05-10 07:21:39 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2015-05-10 07:21:39 +0000 |
commit | 68723c20d2abd8dd0df8aaa69736d469de58446f (patch) | |
tree | 0b8700de55f11944307b5f7579b34d407e7aa92d | |
parent | b8811f47ccf554895b1b48be3f881dc74b4fc3dd (diff) | |
download | redmine-68723c20d2abd8dd0df8aaa69736d469de58446f.tar.gz redmine-68723c20d2abd8dd0df8aaa69736d469de58446f.zip |
Merged r14245 (#19655).
git-svn-id: http://svn.redmine.org/redmine/branches/3.0-stable@14249 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r-- | app/controllers/application_controller.rb | 4 | ||||
-rw-r--r-- | test/functional/sessions_test.rb | 10 |
2 files changed, 7 insertions, 7 deletions
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index e52cc6186..6e2136836 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -63,9 +63,9 @@ class ApplicationController < ActionController::Base if session[:user_id] if session_expired? && !try_to_autologin set_localization(User.active.find_by_id(session[:user_id])) - reset_session + self.logged_user = nil flash[:error] = l(:error_session_expired) - redirect_to signin_url + require_login else session[:atime] = Time.now.utc.to_i end diff --git a/test/functional/sessions_test.rb b/test/functional/sessions_test.rb index 1d75fc8db..9e7032682 100644 --- a/test/functional/sessions_test.rb +++ b/test/functional/sessions_test.rb @@ -56,14 +56,14 @@ class SessionsTest < ActionController::TestCase def test_user_session_without_ctime_should_be_reset_if_lifetime_enabled with_settings :session_lifetime => '720' do get :index, {}, {:user_id => 2} - assert_redirected_to '/login' + assert_redirected_to 'http://test.host/login?back_url=http%3A%2F%2Ftest.host%2F' end end def test_user_session_with_expired_ctime_should_be_reset_if_lifetime_enabled with_settings :session_timeout => '720' do get :index, {}, {:user_id => 2, :atime => 2.days.ago.utc.to_i} - assert_redirected_to '/login' + assert_redirected_to 'http://test.host/login?back_url=http%3A%2F%2Ftest.host%2F' end end @@ -77,14 +77,14 @@ class SessionsTest < ActionController::TestCase def test_user_session_without_atime_should_be_reset_if_timeout_enabled with_settings :session_timeout => '60' do get :index, {}, {:user_id => 2} - assert_redirected_to '/login' + assert_redirected_to 'http://test.host/login?back_url=http%3A%2F%2Ftest.host%2F' end end def test_user_session_with_expired_atime_should_be_reset_if_timeout_enabled with_settings :session_timeout => '60' do get :index, {}, {:user_id => 2, :atime => 4.hours.ago.utc.to_i} - assert_redirected_to '/login' + assert_redirected_to 'http://test.host/login?back_url=http%3A%2F%2Ftest.host%2F' end end @@ -117,7 +117,7 @@ class SessionsTest < ActionController::TestCase with_settings :session_timeout => '60' do get :index, {}, {:user_id => user.id, :atime => 4.hours.ago.utc.to_i} - assert_redirected_to '/login' + assert_redirected_to 'http://test.host/login?back_url=http%3A%2F%2Ftest.host%2F' assert_include "Veuillez vous reconnecter", flash[:error] assert_equal :fr, current_language end |