Merge pull request #118 from lbeder/select-1

Use "SELECT 1" to check for DB connectivity and liveness

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['2.7.7', '3.1.3']
+        ruby: ['3.1.4', '3.2.3', '3.3.0']
 
     name: Ruby ${{ matrix.ruby }}
     steps:

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 12.0.0 (2024-03-23)
+
+- Use "SELECT 1" to check for DB connectivity and liveness (thanks to @jayceekay)
+
 ## 11.3.0 (2024-01-21)
 
 - Unpermitting subdomain (thanks to @shettytejas)

lib/health_monitor/providers/database.rb

@@ -11,7 +11,7 @@ def check!
         failed_databases = []
 
         ActiveRecord::Base.connection_handler.connection_pool_list(:all).each do |cp|
-          cp.connection.check_version
+          cp.connection.execute('SELECT 1')
         rescue Exception
           failed_databases << cp.db_config.name
         end

lib/health_monitor/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HealthMonitor
-  VERSION = '11.3.0'
+  VERSION = '12.0.0'
 end

spec/support/providers.rb

@@ -13,7 +13,7 @@ def stub_cache_failure
   end
 
   def stub_database_failure(database = nil)
-    allow_any_instance_of(ActiveRecord::ConnectionAdapters::SQLite3Adapter).to receive(:check_version) do |instance|
+    allow_any_instance_of(ActiveRecord::ConnectionAdapters::SQLite3Adapter).to receive(:execute) do |instance|
       raise StandardError if !database.present? || instance.pool.db_config.name == database.to_s
     end
   end