rails_failover 0.6.5 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 45afa97c24a3c8f4fac5c74a52eabbf0d06b9f7ad037a76b18bee05875b4f374
-  data.tar.gz: e57ffc9182a9b2f4145c04d4f921bcd79aee3edfec5dedacac2fb6f23d39dc9d
+  metadata.gz: 9b0f3e010743d83bd67b2cce785fde0952ba32c82c8c1ab5fef3aa56e5baa520
+  data.tar.gz: 26cc2466069dfe48b691d8676078b3ed9ee3ce82e7a8d4447fa327436741fd39
 SHA512:
-  metadata.gz: 0ee4ae4e79e17881a49f77711b5f13f301977d67cd177f8d16296cff93c0aea63b9151f235acfde87afaae14f6215840fbb3efb6024a98b59c14454b88705356
-  data.tar.gz: 48d815e482d5b6b92316431bda654709f3049647292c4a4d9b8a6a075086be2e23b12cceed898d07e10f729bc85217aa395521a49ffd1c4baad63927ed938963
+  metadata.gz: 5ce28a0d758dde383537bce455502edfcbf9388ad4c2835edc17b2e259156affc885153f728e39fa66c8c25d3f9dfea34a8f489b663e7738fc092dc6c76dccb8
+  data.tar.gz: 0f0c08446e0e64c5eb6ee972e80e8776e58fbd32877254e50098a17a4d4475db64379d7ad237933b054e5eaf7c54df72785c69253e7d31be1b28d38028fcaf11

data/.github/workflows/ci.yml

@@ -5,8 +5,7 @@ on:
   push:
     branches:
       - master
-    tags:
-      - v*
+      - main
 
 jobs:
   build:
@@ -18,22 +17,19 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby:
-          - 2.6
-          - 2.7
-        build_types: ["LINT", "REDIS", "ACTIVERECORD"]
-        exclude:
-          - ruby: 2.6
+        ruby: ["2.6", "2.7", "3.0", "3.1"]
+        build_types: ["REDIS", "ACTIVERECORD"]
+        include:
+          - ruby: "3.1"
             build_types: "LINT"
 
     steps:
-      - uses: actions/checkout@v1
+      - uses: actions/checkout@v2
 
       - name: Setup ruby
-        uses: actions/setup-ruby@v1
+        uses: ruby/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby }}
-          architecture: 'x64'
 
       - name: Setup bundler
         run: gem install bundler
@@ -68,7 +64,7 @@ jobs:
         if: env.BUILD_TYPE == 'ACTIVERECORD'
 
   publish:
-    if: contains(github.ref, 'refs/tags/v')
+    if: github.event_name == 'push' && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/master')
     needs: build
     runs-on: ubuntu-latest
 
@@ -76,6 +72,8 @@ jobs:
       - uses: actions/checkout@v2
 
       - name: Release Gem
-        uses: CvX/publish-rubygems-action@master
+        uses: discourse/publish-rubygems-action@v2
         env:
-          RUBYGEMS_API_KEY: ${{secrets.RUBYGEMS_API_KEY}}
+          RUBYGEMS_API_KEY: ${{ secrets.RUBYGEMS_API_KEY }}
+          GIT_EMAIL: team@discourse.org
+          GIT_NAME: discoursebot

data/.gitignore

@@ -16,3 +16,4 @@
 *.rdb
 
 *.gem
+Gemfile.lock

data/CHANGELOG.md

@@ -6,6 +6,26 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.8.0] - 2022-01-17
+
+- FEATURE: Compatibility with Rails 7.0+
+
+## [0.7.3] - 2021-04-15
+
+- FEATURE: Compatibility with Rails 6.1
+
+## [0.7.2] - 2021-04-14
+
+No changes.
+
+## [0.7.1] - 2021-04-14
+
+- FIX: Backward compatability with Rails 6.0
+
+## [0.7.0] - 2021-04-14
+
+- FEATURE: Partial compatibility with Rails 6.1
+
 ## [0.6.5] - 2020-12-16
 
 - FIX: Catch exceptions that are not intercepted by `ActionDispatch::DebugExceptions`.

data/README.md

@@ -53,7 +53,7 @@ end
 
 #### Multiple connection handlers
 
-Note: This API is unstable and is likely to changes when Rails 6.1 is released with sharding support.
+Note: This API is unstable and is likely to change when Rails 6.1 is released with sharding support.
 
 ```
 # config/database.yml

data/lib/rails_failover/active_record/handler.rb

@@ -56,13 +56,20 @@ module RailsFailover
 
         primaries_down.keys.each do |handler_key|
           connection_handler = ::ActiveRecord::Base.connection_handlers[handler_key]
-          spec = connection_handler.retrieve_connection_pool(spec_name).spec
-          config = spec.config
+
+          connection_pool = connection_handler.retrieve_connection_pool(spec_name)
+          if connection_pool.respond_to?(:db_config)
+            config = connection_pool.db_config.configuration_hash
+            adapter_method = connection_pool.db_config.adapter_method
+          else
+            config = connection_pool.spec.config
+            adapter_method = connection_pool.spec.adapter_method
+          end
           logger.debug "#{Process.pid} Checking server for '#{handler_key} #{spec_name}'..."
           connection_active = false
 
           begin
-            connection = ::ActiveRecord::Base.public_send(spec.adapter_method, config)
+            connection = ::ActiveRecord::Base.public_send(adapter_method, config)
             connection_active = connection.active?
           rescue => e
             logger.debug "#{Process.pid} Connection to server for '#{handler_key} #{spec_name}' failed with '#{e.message}'"

data/lib/rails_failover/active_record/middleware.rb

@@ -80,7 +80,12 @@ module RailsFailover
           handler = ::ActiveRecord::ConnectionAdapters::ConnectionHandler.new
 
           ::ActiveRecord::Base.connection_handlers[writing_role].connection_pools.each do |pool|
-            ::RailsFailover::ActiveRecord.establish_reading_connection(handler, pool.spec)
+            if pool.respond_to?(:db_config)
+              config = pool.db_config.configuration_hash
+            else
+              config = pool.spec.config
+            end
+            ::RailsFailover::ActiveRecord.establish_reading_connection(handler, config)
           end
 
           handler

data/lib/rails_failover/active_record/railtie.rb

@@ -4,7 +4,15 @@ module RailsFailover
   module ActiveRecord
     class Railtie < ::Rails::Railtie
       initializer "rails_failover.init", after: "active_record.initialize_database" do |app|
-        config = ::ActiveRecord::Base.connection_config
+
+        # AR 6.0 / 6.1 compat
+        config =
+          if ::ActiveRecord::Base.respond_to? :connection_db_config
+            ::ActiveRecord::Base.connection_db_config.configuration_hash
+          else
+            ::ActiveRecord::Base.connection_config
+          end
+
         app.config.active_record_rails_failover = false
 
         if !!(config[:replica_host] && config[:replica_port])
@@ -19,9 +27,14 @@ module RailsFailover
               ::ActiveRecord::ConnectionAdapters::ConnectionHandler.new
 
             ::ActiveRecord::Base.connection_handlers[::ActiveRecord::Base.writing_role].connection_pools.each do |connection_pool|
+              if connection_pool.respond_to?(:db_config)
+                config = connection_pool.db_config.configuration_hash
+              else
+                config = connection_pool.spec.config
+              end
               RailsFailover::ActiveRecord.establish_reading_connection(
                 ::ActiveRecord::Base.connection_handlers[::ActiveRecord::Base.reading_role],
-                connection_pool.spec
+                config
               )
             end
 
@@ -29,7 +42,7 @@ module RailsFailover
               ::ActiveRecord::Base.connection
             rescue ::ActiveRecord::NoDatabaseError
               # Do nothing since database hasn't been created
-            rescue ::PG::Error => e
+            rescue ::PG::Error, ::ActiveRecord::ConnectionNotEstablished
               Handler.instance.verify_primary(::ActiveRecord::Base.writing_role)
               ::ActiveRecord::Base.connection_handler = ::ActiveRecord::Base.lookup_connection_handler(:reading)
             end

data/lib/rails_failover/active_record.rb

@@ -27,8 +27,7 @@ module RailsFailover
       @verify_primary_frequency_seconds || 5
     end
 
-    def self.establish_reading_connection(handler, connection_spec)
-      config = connection_spec.config
+    def self.establish_reading_connection(handler, config)
 
       if config[:replica_host] && config[:replica_port]
         replica_config = config.dup

data/lib/rails_failover/redis/handler.rb

@@ -169,7 +169,7 @@ module RailsFailover
       # Calling .disconnect can cause a running subscribe() to block forever
       # Therefore try to acquire the lock
       def soft_disconnect(redis, client, role)
-        has_lock = redis.mon_try_enter
+        has_lock = redis_mon_try_enter(redis)
 
         if !has_lock
           begin
@@ -180,7 +180,7 @@ module RailsFailover
 
           waiting_since = Process.clock_gettime(Process::CLOCK_MONOTONIC)
           loop do # Keep trying
-            break if has_lock = redis.mon_try_enter
+            break if has_lock = redis_mon_try_enter(redis)
             break if !client.connection.connected? # Disconnected by other thread
             break if client.connection.rails_failover_role != role # Reconnected by other thread
             time_now = Process.clock_gettime(Process::CLOCK_MONOTONIC)
@@ -191,12 +191,28 @@ module RailsFailover
 
         client.disconnect if client.connection&.rails_failover_role == role
       ensure
-        redis.mon_exit if has_lock
+        redis_mon_exit(redis) if has_lock
       end
 
       def logger
         RailsFailover::Redis.logger
       end
+
+      def redis_mon_try_enter(redis)
+        if redis.respond_to? :mon_try_enter
+          redis.mon_try_enter
+        else
+          redis.instance_variable_get(:@monitor).mon_try_enter
+        end
+      end
+
+      def redis_mon_exit(redis)
+        if redis.respond_to? :mon_exit
+          redis.mon_exit
+        else
+          redis.instance_variable_get(:@monitor).mon_exit
+        end
+      end
     end
   end
 end

data/lib/rails_failover/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsFailover
-  VERSION = "0.6.5"
+  VERSION = "0.8.1"
 end

data/rails_failover.gemspec

@@ -9,8 +9,9 @@ Gem::Specification.new do |spec|
   spec.email         = ["tgx@discourse.org"]
 
   spec.summary       = %q{Failover for ActiveRecord and Redis}
+  spec.homepage      = "https://github.com/discourse/rails_failover"
   spec.license       = "MIT"
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.3.0")
+  spec.required_ruby_version = Gem::Requirement.new(">= 2.5.0")
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
@@ -21,8 +22,8 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  ["activerecord", "railties"].each do |gem_name|
-    spec.add_dependency gem_name, "~> 6.0"
-  end
+  spec.add_dependency "activerecord", "> 6.0", "< 7.1"
+  spec.add_dependency "railties", "> 6.0", "< 7.1"
+
   spec.add_dependency "concurrent-ruby"
 end

metadata

@@ -1,43 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: rails_failover
 version: !ruby/object:Gem::Version
-  version: 0.6.5
+  version: 0.8.1
 platform: ruby
 authors:
 - Alan Tan
-autorequire:
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-12-16 00:00:00.000000000 Z
+date: 2022-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '6.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '6.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '6.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '6.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.1'
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
   requirement: !ruby/object:Gem::Requirement
@@ -52,7 +64,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description:
+description: 
 email:
 - tgx@discourse.org
 executables: []
@@ -66,7 +78,6 @@ files:
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
-- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -87,11 +98,11 @@ files:
 - postgresql.mk
 - rails_failover.gemspec
 - redis.mk
-homepage:
+homepage: https://github.com/discourse/rails_failover
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -99,15 +110,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key:
+rubygems_version: 3.1.6
+signing_key: 
 specification_version: 4
 summary: Failover for ActiveRecord and Redis
 test_files: []

data/Gemfile.lock

@@ -1,123 +0,0 @@
-PATH
-  remote: .
-  specs:
-    rails_failover (0.6.5)
-      activerecord (~> 6.0)
-      concurrent-ruby
-      railties (~> 6.0)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    actionpack (6.0.3.1)
-      actionview (= 6.0.3.1)
-      activesupport (= 6.0.3.1)
-      rack (~> 2.0, >= 2.0.8)
-      rack-test (>= 0.6.3)
-      rails-dom-testing (~> 2.0)
-      rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actionview (6.0.3.1)
-      activesupport (= 6.0.3.1)
-      builder (~> 3.1)
-      erubi (~> 1.4)
-      rails-dom-testing (~> 2.0)
-      rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activemodel (6.0.3.1)
-      activesupport (= 6.0.3.1)
-    activerecord (6.0.3.1)
-      activemodel (= 6.0.3.1)
-      activesupport (= 6.0.3.1)
-    activesupport (6.0.3.1)
-      concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.2, >= 2.2.2)
-    ast (2.4.0)
-    builder (3.2.4)
-    byebug (11.1.3)
-    concurrent-ruby (1.1.6)
-    crass (1.0.6)
-    diff-lcs (1.3)
-    erubi (1.10.0)
-    i18n (1.8.2)
-      concurrent-ruby (~> 1.0)
-    loofah (2.7.0)
-      crass (~> 1.0.2)
-      nokogiri (>= 1.5.9)
-    method_source (1.0.0)
-    mini_portile2 (2.4.0)
-    minitest (5.14.1)
-    nokogiri (1.10.10)
-      mini_portile2 (~> 2.4.0)
-    parallel (1.19.1)
-    parser (2.7.1.2)
-      ast (~> 2.4.0)
-    pg (1.2.3)
-    rack (2.2.3)
-    rack-test (1.1.0)
-      rack (>= 1.0, < 3)
-    rails-dom-testing (2.0.3)
-      activesupport (>= 4.2.0)
-      nokogiri (>= 1.6)
-    rails-html-sanitizer (1.3.0)
-      loofah (~> 2.3)
-    railties (6.0.3.1)
-      actionpack (= 6.0.3.1)
-      activesupport (= 6.0.3.1)
-      method_source
-      rake (>= 0.8.7)
-      thor (>= 0.20.3, < 2.0)
-    rainbow (3.0.0)
-    rake (12.3.3)
-    redis (4.1.4)
-    rexml (3.2.4)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.2)
-      rspec-support (~> 3.9.3)
-    rspec-expectations (3.9.2)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.3)
-    rubocop (0.83.0)
-      parallel (~> 1.10)
-      parser (>= 2.7.0.1)
-      rainbow (>= 2.2.2, < 4.0)
-      rexml
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 2.0)
-    rubocop-discourse (2.1.2)
-      rubocop (>= 0.69.0)
-      rubocop-rspec (>= 1.39.0)
-    rubocop-rspec (1.39.0)
-      rubocop (>= 0.68.1)
-    ruby-progressbar (1.10.1)
-    thor (1.0.1)
-    thread_safe (0.3.6)
-    tzinfo (1.2.7)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.7.0)
-    zeitwerk (2.3.0)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  activerecord (~> 6.0)
-  byebug
-  pg (~> 1.2)
-  rack
-  rails_failover!
-  rake (~> 12.0)
-  redis (~> 4.1)
-  rspec (~> 3.0)
-  rubocop-discourse
-
-BUNDLED WITH
-   2.1.4