FIX: Race-condition in fallback handlers (#8005)

Calling `verify_master` in multiple threads simultaneously would cause
multiple threads to be spawned.

lib/active_record/connection_adapters/postgresql_fallback_adapter.rb

@@ -32,19 +32,21 @@ class PostgreSQLFallbackHandler
   end
 
   def verify_master
-    synchronize { return if @thread && @thread.alive? }
+    synchronize do
+      return if @thread && @thread.alive?
 
-    @thread = Thread.new do
+      @thread = Thread.new do
-      while true do
+        while true do
-        thread = Thread.new { initiate_fallback_to_master }
+          thread = Thread.new { initiate_fallback_to_master }
-        thread.abort_on_exception = true
+          thread.abort_on_exception = true
-        thread.join
+          thread.join
-        break if synchronize { @masters_down.hash.empty? }
+          break if synchronize { @masters_down.hash.empty? }
-        sleep 5
+          sleep 5
+        end
       end
-    end
 
-    @thread.abort_on_exception = true
+      @thread.abort_on_exception = true
+    end
   end
 
   def master_down?

lib/discourse_redis.rb

@@ -23,17 +23,19 @@ class DiscourseRedis
     end
 
     def verify_master
-      synchronize { return if @thread && @thread.alive? }
+      synchronize do
+        return if @thread && @thread.alive?
 
-      @thread = Thread.new do
+        @thread = Thread.new do
-        loop do
+          loop do
-          begin
+            begin
-            thread = Thread.new { initiate_fallback_to_master }
+              thread = Thread.new { initiate_fallback_to_master }
-            thread.join
+              thread.join
-            break if synchronize { @master }
+              break if synchronize { @master }
-            sleep 5
+              sleep 5
-          ensure
+            ensure
-            thread.kill
+              thread.kill
+            end
           end
         end
       end