X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/f18baae22e4088af7f252c82f3c4a2576862a438..01cbc40ed62f006398bcce62fc1f449b432b59b1:/app/controllers/sessions_controller.rb?ds=sidebyside diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb index 130471670..3c2084a5b 100644 --- a/app/controllers/sessions_controller.rb +++ b/app/controllers/sessions_controller.rb @@ -4,14 +4,15 @@ class SessionsController < ApplicationController layout "site" before_action :disable_terms_redirect, :only => [:destroy] + before_action :authorize_web + before_action :set_locale + before_action :check_database_readable before_action :require_cookies, :only => [:new] 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 @@ -19,11 +20,11 @@ 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] @@ -31,13 +32,13 @@ class SessionsController < ApplicationController token&.destroy session.delete(:token) end + 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 @@ -51,7 +52,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