From 0babc034306ce7cc9620eaba3cbf9f236f0648d5 Mon Sep 17 00:00:00 2001
From: James Paterni <james@blueleaf.com>
Date: Mon, 14 Jul 2025 23:22:16 -0400
Subject: [PATCH] Fix method visibility in BackupMonitor for health check
 endpoints
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Move get_recent_backups, get_failed_backups, and get_container_backup_history methods to public section
- Remove duplicate methods that were incorrectly placed in private section
- Health check server endpoints now fully functional

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 app/lib/baktainer/backup_monitor.rb | 46 ++++++++++++++---------------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/app/lib/baktainer/backup_monitor.rb b/app/lib/baktainer/backup_monitor.rb
index 71fb62f..f625fa7 100644
--- a/app/lib/baktainer/backup_monitor.rb
+++ b/app/lib/baktainer/backup_monitor.rb
@@ -122,6 +122,29 @@ class Baktainer::BackupMonitor
     @logger.info("Cleared all performance alerts")
   end
 
+  # Get recent backups for health check endpoints
+  def get_recent_backups(limit = 50)
+    @mutex.synchronize do
+      @backup_history.last(limit).reverse
+    end
+  end
+
+  # Get failed backups for health check endpoints  
+  def get_failed_backups(limit = 20)
+    @mutex.synchronize do
+      failed = @backup_history.select { |b| b[:status] == 'failed' }
+      failed.last(limit).reverse
+    end
+  end
+
+  # Get backup history for a specific container
+  def get_container_backup_history(container_name, limit = 20)
+    @mutex.synchronize do
+      container_backups = @backup_history.select { |b| b[:container_name] == container_name }
+      container_backups.last(limit).reverse
+    end
+  end
+
   def export_metrics(format = :json)
     case format
     when :json
@@ -230,27 +253,4 @@ class Baktainer::BackupMonitor
       end
     end
   end
-
-  # Get recent backups for health check endpoints
-  def get_recent_backups(limit = 50)
-    @mutex.synchronize do
-      @backup_history.last(limit).reverse
-    end
-  end
-
-  # Get failed backups for health check endpoints  
-  def get_failed_backups(limit = 20)
-    @mutex.synchronize do
-      failed = @backup_history.select { |b| b[:status] == 'failed' }
-      failed.last(limit).reverse
-    end
-  end
-
-  # Get backup history for a specific container
-  def get_container_backup_history(container_name, limit = 20)
-    @mutex.synchronize do
-      container_backups = @backup_history.select { |b| b[:container_name] == container_name }
-      container_backups.last(limit).reverse
-    end
-  end
 end
\ No newline at end of file