Make a generic preferred color scheme method

[rails.git] / app / controllers / sessions_controller.rb

diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb
index 13047167075245b17ee6f09353a14c81d9ef5464..abbaf5e921e45aedf89e60d7057023b9a0374143 100644 (file)
--- a/app/controllers/sessions_controller.rb
+++ b/app/controllers/sessions_controller.rb
@@ -4,40 +4,40 @@ class SessionsController < ApplicationController
   layout "site"
 
   before_action :disable_terms_redirect, :only => [:destroy]
   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
 
   before_action :require_cookies, :only => [:new]
 
   authorize_resource :class => false
 

+  allow_all_form_action :only => :new
+

   def new
   def new

-    append_content_security_policy_directives(
-      :form_action => %w[*]
-    )
+    referer = safe_referer(params[:referer]) if params[:referer]

 
 

-    session[:referer] = safe_referer(params[:referer]) if params[:referer]
+    parse_oauth_referer referer

   end
 
   def create
   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
   end
 
   def destroy

-    @title = t "sessions.destroy.title"
+    @title = t ".title"

 
     if request.post?
 
     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
       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
 
     end
   end
 


@@ -45,15 +45,15 @@ class SessionsController < ApplicationController
 
   ##
   # handle password authentication
 
   ##
   # handle password authentication

-  def password_authentication(username, password)
+  def password_authentication(username, password, referer = nil)

     if (user = User.authenticate(:username => username, :password => password))
     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))
     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)
     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
     else

-      failed_login t("sessions.new.auth failure"), username
+      failed_login(t("sessions.new.auth failure"), username, referer)

     end
   end
 end
     end
   end
 end