Ruby 3 Patch

Author: ishandeveloper

lib/action_dispatch/session/active_record_store.rb

@@ -58,15 +58,11 @@ class ActiveRecordStore < ActionDispatch::Session::AbstractStore
       cattr_accessor :session_class
 
       SESSION_RECORD_KEY = 'rack.session.record'
-      if Rack.const_defined?(:RACK_SESSION_OPTIONS)
-        ENV_SESSION_OPTIONS_KEY = Rack::RACK_SESSION_OPTIONS
-      else
-        ENV_SESSION_OPTIONS_KEY = Rack::Session::Abstract::ENV_SESSION_OPTIONS_KEY
-      end
+      ENV_SESSION_OPTIONS_KEY = Rack::RACK_SESSION_OPTIONS
 
     private
       def get_session(request, sid)
-        logger.silence_logger do
+        logger.silence do
           unless sid and session = @@session_class.find_by_session_id(sid)
             # If the sid was nil or if there is no pre-existing session under the sid,
             # force the generation of a new sid and associate a new session associated with the new sid
@@ -79,7 +75,7 @@ def get_session(request, sid)
       end
 
       def write_session(request, sid, session_data, options)
-        logger.silence_logger do
+        logger.silence do
           record = get_session_model(request, sid)
           record.data = session_data
           return false unless record.save
@@ -96,7 +92,7 @@ def write_session(request, sid, session_data, options)
       end
 
       def delete_session(request, session_id, options)
-        logger.silence_logger do
+        logger.silence do
           if sid = current_session_id(request)
             if model = @@session_class.find_by_session_id(sid)
               data = model.data
@@ -120,7 +116,7 @@ def delete_session(request, session_id, options)
       end
 
       def get_session_model(request, id)
-        logger.silence_logger do
+        logger.silence do
           model = @@session_class.find_by_session_id(id)
           if !model
             id = generate_sid
@@ -141,14 +137,15 @@ def find_session(request, id)
         [model.session_id, model.data]
       end
 
+      module NilLogger
+        def self.silence
+          yield
+        end
+      end
+
       def logger
-        ActiveRecord::Base.logger || ActiveRecord::SessionStore::NilLogger
+        ActiveRecord::Base.logger || NilLogger
       end
     end
   end
-end
-
-if ActiveRecord::VERSION::MAJOR == 4
-  require 'action_dispatch/session/legacy_support'
-  ActionDispatch::Session::ActiveRecordStore.send(:include, ActionDispatch::Session::LegacySupport)
-end
+end

lib/active_record/session_store.rb

@@ -1,7 +1,6 @@
 require 'active_record'
 require 'active_record/session_store/version'
 require 'action_dispatch/session/active_record_store'
-require "active_record/session_store/extension/logger_silencer"
 require 'active_support/core_ext/hash/keys'
 require 'multi_json'
 
@@ -116,11 +115,4 @@ def self.dump(value)
 end
 
 require 'active_record/session_store/sql_bypass'
-require 'active_record/session_store/railtie' if defined?(Rails)
-
-Logger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer
-
-begin
-  require "syslog/logger"
-  Syslog::Logger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer
-rescue LoadError; end
+require 'active_record/session_store/railtie' if defined?(Rails)

lib/active_record/session_store/extension/logger_silencer.rb

@@ -10,11 +10,7 @@ def set_session_value
     end
 
     def get_session_value
-      if ActiveRecord::VERSION::MAJOR == 4
-        render :text => "foo: #{session[:foo].inspect}"
-      else
-        render :plain => "foo: #{session[:foo].inspect}"
-      end
+      render :plain => "foo: #{session[:foo].inspect}"
     end
   end
 
@@ -41,25 +37,13 @@ def setup
   end
 
   def test_log_silencer_with_logger_not_raise_exception
-    with_logger Logger.new(Tempfile.new("tempfile")) do
+    with_logger ActiveSupport::Logger.new(Tempfile.new("tempfile")) do
       with_test_route_set do
         get "/set_session_value"
       end
     end
   end
 
-  begin
-    require "syslogger/logger"
-
-    def test_log_silencer_with_syslog_logger_not_raise_exception
-      with_logger Syslog::Logger.new("ar_session_store_test") do
-        with_test_route_set do
-          get "/set_session_value"
-        end
-      end
-    end
-  rescue LoadError; end
-
   private
 
     def with_logger(logger)
@@ -71,10 +55,10 @@ def with_logger(logger)
     end
 
     def with_fake_logger(&block)
-      with_logger(Logger.new(fake_logger), &block)
+      with_logger(ActiveSupport::Logger.new(fake_logger), &block)
     end
 
     def fake_logger
       @fake_logger ||= StringIO.new
     end
-end
+end