FIX: Force ActiveRecord reading role if Redis is down take 2.

FIX: Force ActiveRecord reading role if Redis is down take 2.

follow-up f03c7a1ba1cf

diff --git a/config/initializers/002-rails_failover.rb b/config/initializers/002-rails_failover.rb
index 7b948c0..99def93 100644
--- a/config/initializers/002-rails_failover.rb
+++ b/config/initializers/002-rails_failover.rb
@@ -55,10 +55,10 @@ if defined?(RailsFailover::ActiveRecord)
       Discourse::PG_FORCE_READONLY_MODE_KEY
     )
   rescue => e
-    unless e.is_a?(Redis::CannotConnectError)
-      Rails.logger.warn "#{e.class} #{e.message}: #{e.backtrace.join("\n")}"
+    if e.is_a?(Redis::CannotConnectError)
       true
     else
+      Rails.logger.warn "#{e.class} #{e.message}: #{e.backtrace.join("\n")}"
       false
     end
   end

GitHub sha: fd38c2fa

1 Like