X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/41f95466faf0cdf3aed3be70f88f0a23d17b420e..71cb2c50fcaf29e2b933fc981e40a00223d94d34:/app/controllers/sessions_controller.rb diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb index 7e6a740f3..e57ffc06a 100644 --- a/app/controllers/sessions_controller.rb +++ b/app/controllers/sessions_controller.rb @@ -12,9 +12,7 @@ class SessionsController < ApplicationController authorize_resource :class => false def new - append_content_security_policy_directives( - :form_action => %w[*] - ) + override_content_security_policy_directives(:form_action => []) if Settings.csp_enforce || Settings.key?(:csp_report_url) session[:referer] = safe_referer(params[:referer]) if params[:referer] end @@ -22,25 +20,20 @@ class SessionsController < ApplicationController def create session[:remember_me] ||= params[:remember_me] session[:referer] = safe_referer(params[:referer]) if params[:referer] - password_authentication(params[:username], params[:password]) + password_authentication(params[:username].strip, params[:password]) 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 - if params[:referer] - redirect_to safe_referer(params[:referer]) - else - redirect_to :controller => "site", :action => "index" - end + + referer = safe_referer(params[:referer]) if params[:referer] + + redirect_to referer || { :controller => "site", :action => "index" } end end @@ -54,7 +47,7 @@ class SessionsController < ApplicationController elsif (user = User.authenticate(:username => username, :password => password, :pending => true)) unconfirmed_login(user) 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) else failed_login t("sessions.new.auth failure"), username end