jro_apps/redmine Commit - r2966:0485d3a52408

Reset session on login/logout (#4248)....

Jean-Philippe Lang -

r2966:0485d3a52408

parent child

Context file:

r2966:0485d3a52408

Collapse all files

app/controllers/application_controller.rb +1 -1

              # redMine - project management software
              # Copyright (C) 2006-2007  Jean-Philippe Lang
              #
              # This program is free software; you can redistribute it and/or
              # modify it under the terms of the GNU General Public License
              # as published by the Free Software Foundation; either version 2
              # of the License, or (at your option) any later version.
              #
              # This program is distributed in the hope that it will be useful,
              # but WITHOUT ANY WARRANTY; without even the implied warranty of
              # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
              # GNU General Public License for more details.
              #
              # You should have received a copy of the GNU General Public License
              # along with this program; if not, write to the Free Software
              # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
              require 'uri'
              require 'cgi'
              class ApplicationController < ActionController::Base
                include Redmine::I18n
                layout 'base'
                before_filter :user_setup, :check_if_login_required, :set_localization
                filter_parameter_logging :password
                protect_from_forgery
                include Redmine::Search::Controller
                include Redmine::MenuManager::MenuController
                helper Redmine::MenuManager::MenuHelper
                REDMINE_SUPPORTED_SCM.each do |scm|
                  require_dependency "repository/#{scm.underscore}"
                end
                def user_setup
                  # Check the settings cache for each request
                  Setting.check_cache
                  # Find the current user
                  User.current = find_current_user
                end
                # Returns the current user or nil if no user is logged in
                # and starts a session if needed
                def find_current_user
                  if session[:user_id]
                    # existing session
                    (User.active.find(session[:user_id]) rescue nil)
                  elsif cookies[:autologin] && Setting.autologin?
                    # auto-login feature starts a new session
                    user = User.try_to_autologin(cookies[:autologin])
                    session[:user_id] = user.id if user
                    user
                  elsif params[:format] == 'atom' && params[:key] && accept_key_auth_actions.include?(params[:action])
                    # RSS key authentication does not start a session
                    User.find_by_rss_key(params[:key])
                  end
                end
                # Sets the logged in user
                def logged_user=(user)
+                 reset_session
                  if user && user.is_a?(User)
                    User.current = user
                    session[:user_id] = user.id
                  else
                    User.current = User.anonymous
-                   session[:user_id] = nil
                  end
                end
                # check if login is globally required to access the application
                def check_if_login_required
                  # no check needed if user is already logged in
                  return true if User.current.logged?
                  require_login if Setting.login_required?
                end
                def set_localization
                  lang = nil
                  if User.current.logged?
                    lang = find_language(User.current.language)
                  end
                  if lang.nil? && request.env['HTTP_ACCEPT_LANGUAGE']
                    accept_lang = parse_qvalues(request.env['HTTP_ACCEPT_LANGUAGE']).first.downcase
                    if !accept_lang.blank?
                      lang = find_language(accept_lang) || find_language(accept_lang.split('-').first)
                    end
                  end
                  lang ||= Setting.default_language
                  set_language_if_valid(lang)
                end
                def require_login
                  if !User.current.logged?
                    # Extract only the basic url parameters on non-GET requests
                    if request.get?
                      url = url_for(params)
                    else
                      url = url_for(:controller => params[:controller], :action => params[:action], :id => params[:id], :project_id => params[:project_id])
                    end
                    redirect_to :controller => "account", :action => "login", :back_url => url
                    return false
                  end
                  true
                end
                def require_admin
                  return unless require_login
                  if !User.current.admin?
                    render_403
                    return false
                  end
                  true
                end
                def deny_access
                  User.current.logged? ? render_403 : require_login
                end
                # Authorize the user for the requested action
                def authorize(ctrl = params[:controller], action = params[:action], global = false)
                  allowed = User.current.allowed_to?({:controller => ctrl, :action => action}, @project, :global => global)
                  allowed ? true : deny_access
                end
                # Authorize the user for the requested action outside a project
                def authorize_global(ctrl = params[:controller], action = params[:action], global = true)
                  authorize(ctrl, action, global)
                end
                # make sure that the user is a member of the project (or admin) if project is private
                # used as a before_filter for actions that do not require any particular permission on the project
                def check_project_privacy
                  if @project && @project.active?
                    if @project.is_public? || User.current.member_of?(@project) || User.current.admin?
                      true
                    else
                      User.current.logged? ? render_403 : require_login
                    end
                  else
                    @project = nil
                    render_404
                    false
                  end
                end
                def redirect_back_or_default(default)
                  back_url = CGI.unescape(params[:back_url].to_s)
                  if !back_url.blank?
                    begin
                      uri = URI.parse(back_url)
                      # do not redirect user to another host or to the login or register page
                      if (uri.relative? || (uri.host == request.host)) && !uri.path.match(%r{/(login|account/register)})
                        redirect_to(back_url) and return
                      end
                    rescue URI::InvalidURIError
                      # redirect to default
                    end
                  end
                  redirect_to default
                end
                def render_403
                  @project = nil
                  render :template => "common/403", :layout => !request.xhr?, :status => 403
                  return false
                end
                def render_404
                  render :template => "common/404", :layout => !request.xhr?, :status => 404
                  return false
                end
                def render_error(msg)
                  flash.now[:error] = msg
                  render :text => '', :layout => !request.xhr?, :status => 500
                end
                def render_feed(items, options={})
                  @items = items || []
                  @items.sort! {|x,y| y.event_datetime <=> x.event_datetime }
                  @items = @items.slice(0, Setting.feeds_limit.to_i)
                  @title = options[:title] || Setting.app_title
                  render :template => "common/feed.atom.rxml", :layout => false, :content_type => 'application/atom+xml'
                end
                def self.accept_key_auth(*actions)
                  actions = actions.flatten.map(&:to_s)
                  write_inheritable_attribute('accept_key_auth_actions', actions)
                end
                def accept_key_auth_actions
                  self.class.read_inheritable_attribute('accept_key_auth_actions') || []
                end
                # TODO: move to model
                def attach_files(obj, attachments)
                  attached = []
                  unsaved = []
                  if attachments && attachments.is_a?(Hash)
                    attachments.each_value do |attachment|
                      file = attachment['file']
                      next unless file && file.size > 0
                      a = Attachment.create(:container => obj,
                                            :file => file,
                                            :description => attachment['description'].to_s.strip,
                                            :author => User.current)
                      a.new_record? ? (unsaved << a) : (attached << a)
                    end
                    if unsaved.any?
                      flash[:warning] = l(:warning_attachments_not_saved, unsaved.size)
                    end
                  end
                  attached
                end
                # Returns the number of objects that should be displayed
                # on the paginated list
                def per_page_option
                  per_page = nil
                  if params[:per_page] && Setting.per_page_options_array.include?(params[:per_page].to_s.to_i)
                    per_page = params[:per_page].to_s.to_i
                    session[:per_page] = per_page
                  elsif session[:per_page]
                    per_page = session[:per_page]
                  else
                    per_page = Setting.per_page_options_array.first || 25
                  end
                  per_page
                end
                # qvalues http header parser
                # code taken from webrick
                def parse_qvalues(value)
                  tmp = []
                  if value
                    parts = value.split(/,\s*/)
                    parts.each {|part|
                      if m = %r{^([^\s,]+?)(?:;\s*q=(\d+(?:\.\d+)?))?$}.match(part)
                        val = m[1]
                        q = (m[2] or 1).to_f
                        tmp.push([val, q])
                      end
                    }
                    tmp = tmp.sort_by{|val, q| -q}
                    tmp.collect!{|val, q| val}
                  end
                  return tmp
                rescue
                  nil
                end
                # Returns a string that can be used as filename value in Content-Disposition header
                def filename_for_content_disposition(name)
                  request.env['HTTP_USER_AGENT'] =~ %r{MSIE} ? ERB::Util.url_encode(name) : name
                end
              end

test/integration/account_test.rb +18 0

              # redMine - project management software
              # Copyright (C) 2006-2007  Jean-Philippe Lang
              #
              # This program is free software; you can redistribute it and/or
              # modify it under the terms of the GNU General Public License
              # as published by the Free Software Foundation; either version 2
              # of the License, or (at your option) any later version.
              #
              # This program is distributed in the hope that it will be useful,
              # but WITHOUT ANY WARRANTY; without even the implied warranty of
              # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
              # GNU General Public License for more details.
              #
              # You should have received a copy of the GNU General Public License
              # along with this program; if not, write to the Free Software
              # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
              require "#{File.dirname(__FILE__)}/../test_helper"
              begin
                require 'mocha'
              rescue
                # Won't run some tests
              end
              class AccountTest < ActionController::IntegrationTest
                fixtures :users
                # Replace this with your real tests.
                def test_login
                  get "my/page"
                  assert_redirected_to "/login?back_url=http%3A%2F%2Fwww.example.com%2Fmy%2Fpage"
                  log_user('jsmith', 'jsmith')
                  get "my/account"
                  assert_response :success
                  assert_template "my/account"
                end
                def test_autologin
                  user = User.find(1)
                  Setting.autologin = "7"
                  Token.delete_all
                  # User logs in with 'autologin' checked
                  post '/login', :username => user.login, :password => 'admin', :autologin => 1
                  assert_redirected_to 'my/page'
                  token = Token.find :first
                  assert_not_nil token
                  assert_equal user, token.user
                  assert_equal 'autologin', token.action
                  assert_equal user.id, session[:user_id]
                  assert_equal token.value, cookies['autologin']
                  # Session is cleared
                  reset!
                  User.current = nil
                  # Clears user's last login timestamp
                  user.update_attribute :last_login_on, nil
                  assert_nil user.reload.last_login_on
                  # User comes back with his autologin cookie
                  cookies[:autologin] = token.value
                  get '/my/page'
                  assert_response :success
                  assert_template 'my/page'
                  assert_equal user.id, session[:user_id]
                  assert_not_nil user.reload.last_login_on
                  assert user.last_login_on.utc > 10.second.ago.utc
                end
                def test_lost_password
                  Token.delete_all
                  get "account/lost_password"
                  assert_response :success
                  assert_template "account/lost_password"
                  post "account/lost_password", :mail => 'jSmith@somenet.foo'
                  assert_redirected_to "/login"
                  token = Token.find(:first)
                  assert_equal 'recovery', token.action
                  assert_equal 'jsmith@somenet.foo', token.user.mail
                  assert !token.expired?
                  get "account/lost_password", :token => token.value
                  assert_response :success
                  assert_template "account/password_recovery"
                  post "account/lost_password", :token => token.value, :new_password => 'newpass', :new_password_confirmation => 'newpass'
                  assert_redirected_to "/login"
                  assert_equal 'Password was successfully updated.', flash[:notice]
                  log_user('jsmith', 'newpass')
                  assert_equal 0, Token.count
                end
                def test_register_with_automatic_activation
                  Setting.self_registration = '3'
                  get 'account/register'
                  assert_response :success
                  assert_template 'account/register'
                  post 'account/register', :user => {:login => "newuser", :language => "en", :firstname => "New", :lastname => "User", :mail => "newuser@foo.bar"},
                                           :password => "newpass", :password_confirmation => "newpass"
                  assert_redirected_to 'my/account'
                  follow_redirect!
                  assert_response :success
                  assert_template 'my/account'
                  user = User.find_by_login('newuser')
                  assert_not_nil user
                  assert user.active?
                  assert_not_nil user.last_login_on
                end
                def test_register_with_manual_activation
                  Setting.self_registration = '2'
                  post 'account/register', :user => {:login => "newuser", :language => "en", :firstname => "New", :lastname => "User", :mail => "newuser@foo.bar"},
                                           :password => "newpass", :password_confirmation => "newpass"
                  assert_redirected_to '/login'
                  assert !User.find_by_login('newuser').active?
                end
                def test_register_with_email_activation
                  Setting.self_registration = '1'
                  Token.delete_all
                  post 'account/register', :user => {:login => "newuser", :language => "en", :firstname => "New", :lastname => "User", :mail => "newuser@foo.bar"},
                                           :password => "newpass", :password_confirmation => "newpass"
                  assert_redirected_to '/login'
                  assert !User.find_by_login('newuser').active?
                  token = Token.find(:first)
                  assert_equal 'register', token.action
                  assert_equal 'newuser@foo.bar', token.user.mail
                  assert !token.expired?
                  get 'account/activate', :token => token.value
                  assert_redirected_to '/login'
                  log_user('newuser', 'newpass')
                end
                if Object.const_defined?(:Mocha)
                def test_onthefly_registration
                  # disable registration
                  Setting.self_registration = '0'
                  AuthSource.expects(:authenticate).returns([:login => 'foo', :firstname => 'Foo', :lastname => 'Smith', :mail => 'foo@bar.com', :auth_source_id => 66])
                  post 'account/login', :username => 'foo', :password => 'bar'
                  assert_redirected_to 'my/page'
                  user = User.find_by_login('foo')
                  assert user.is_a?(User)
                  assert_equal 66, user.auth_source_id
                  assert user.hashed_password.blank?
                end
                def test_onthefly_registration_with_invalid_attributes
                  # disable registration
                  Setting.self_registration = '0'
                  AuthSource.expects(:authenticate).returns([:login => 'foo', :lastname => 'Smith', :auth_source_id => 66])
                  post 'account/login', :username => 'foo', :password => 'bar'
                  assert_response :success
                  assert_template 'account/register'
                  assert_tag :input, :attributes => { :name => 'user[firstname]', :value => '' }
                  assert_tag :input, :attributes => { :name => 'user[lastname]', :value => 'Smith' }
                  assert_no_tag :input, :attributes => { :name => 'user[login]' }
                  assert_no_tag :input, :attributes => { :name => 'user[password]' }
                  post 'account/register', :user => {:firstname => 'Foo', :lastname => 'Smith', :mail => 'foo@bar.com'}
                  assert_redirected_to '/my/account'
                  user = User.find_by_login('foo')
                  assert user.is_a?(User)
                  assert_equal 66, user.auth_source_id
                  assert user.hashed_password.blank?
                end
+               def test_login_and_logout_should_clear_session
+                 get '/login'
+                 sid = session[:session_id]
+                 post '/login', :username => 'admin', :password => 'admin'
+                 assert_redirected_to 'my/page'
+                 assert_not_equal sid, session[:session_id], "login should reset session"
+                 assert_equal 1, session[:user_id]
+                 sid = session[:session_id]
+                 get '/'
+                 assert_equal sid, session[:session_id]
+                 get '/logout'
+                 assert_not_equal sid, session[:session_id], "logout should reset session"
+                 assert_nil session[:user_id]
+               end
                else
                  puts 'Mocha is missing. Skipping tests.'
                end
              end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages