X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/446837c3511bf76f11b0d1e6d5a3ff79b69b0d70..79c306ad06155549ff6e5929ff20a1149b4862c4:/app/controllers/sessions_controller.rb?ds=sidebyside diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb index 5b46d67d0..abbaf5e92 100644 --- a/app/controllers/sessions_controller.rb +++ b/app/controllers/sessions_controller.rb @@ -11,38 +11,33 @@ class SessionsController < ApplicationController authorize_resource :class => false + allow_all_form_action :only => :new + def new - override_content_security_policy_directives(:form_action => []) if Settings.csp_enforce || Settings.key?(:csp_report_url) + referer = safe_referer(params[:referer]) if params[:referer] - session[:referer] = safe_referer(params[:referer]) if params[:referer] + parse_oauth_referer referer end def create - session[:remember_me] ||= params[:remember_me] - session[:referer] = safe_referer(params[:referer]) if params[:referer] - password_authentication(params[:username], params[:password]) + session[:remember_me] = params[:remember_me] == "yes" + + referer = safe_referer(params[:referer]) if params[:referer] + + password_authentication(params[:username].strip, params[:password], referer) end def destroy - @title = t "sessions.destroy.title" + @title = t ".title" if request.post? - if session[:token] - token = UserToken.find_by(:token => session[:token]) - token&.destroy - session.delete(:token) - end - + session.delete(:pending_user) session.delete(:user) session_expires_automatically referer = safe_referer(params[:referer]) if params[:referer] - if referer - redirect_to referer - else - redirect_to :controller => "site", :action => "index" - end + redirect_to referer || { :controller => "site", :action => "index" } end end @@ -50,15 +45,15 @@ class SessionsController < ApplicationController ## # handle password authentication - def password_authentication(username, password) + def password_authentication(username, password, referer = nil) if (user = User.authenticate(:username => username, :password => password)) - successful_login(user) + successful_login(user, referer) elsif (user = User.authenticate(:username => username, :password => password, :pending => true)) - unconfirmed_login(user) + unconfirmed_login(user, referer) elsif User.authenticate(:username => username, :password => password, :suspended => true) - failed_login t("sessions.new.account is suspended", :webmaster => "mailto:#{Settings.support_email}").html_safe, username + failed_login({ :partial => "sessions/suspended_flash" }, username, referer) else - failed_login t("sessions.new.auth failure"), username + failed_login(t("sessions.new.auth failure"), username, referer) end end end