From 5e0f008b651c2315ea0782350cabf1fdac1e53b8 Mon Sep 17 00:00:00 2001
From: Shaun Walker <shaun.walker@siliqon.com>
Date: Sat, 8 Jan 2022 10:22:05 -0500
Subject: [PATCH] add error handling in purge job

---
 Oqtane.Server/Infrastructure/Jobs/PurgeJob.cs | 26 +++++++++++++++----
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/Oqtane.Server/Infrastructure/Jobs/PurgeJob.cs b/Oqtane.Server/Infrastructure/Jobs/PurgeJob.cs
index f931ae8b..a73bef98 100644
--- a/Oqtane.Server/Infrastructure/Jobs/PurgeJob.cs
+++ b/Oqtane.Server/Infrastructure/Jobs/PurgeJob.cs
@@ -37,18 +37,27 @@ namespace Oqtane.Infrastructure
             foreach (Site site in sites)
             {
                 log += "Processing Site: " + site.Name + "<br />";
+                int retention;
+                int count;
 
                 // get site settings
                 Dictionary<string, string> settings = GetSettings(settingRepository.GetSettings(EntityNames.Site, site.SiteId).ToList());
 
                 // purge event log
-                int retention = 30; // 30 days
+                retention = 30; // 30 days
                 if (settings.ContainsKey("LogRetention") && !string.IsNullOrEmpty(settings["LogRetention"]))
                 {
                     retention = int.Parse(settings["LogRetention"]);
                 }
-                int count = logRepository.DeleteLogs(retention);
-                log += count.ToString() + " Events Purged<br />";
+                try
+                {
+                    count = logRepository.DeleteLogs(retention);
+                    log += count.ToString() + " Events Purged<br />";
+                }
+                catch (Exception ex)
+                {
+                    log += $"Error Purging Events - {ex.Message}<br />";
+                }
 
                 // purge visitors
                 if (site.VisitorTracking)
@@ -58,8 +67,15 @@ namespace Oqtane.Infrastructure
                     {
                         retention = int.Parse(settings["VisitorRetention"]);
                     }
-                    count = visitorRepository.DeleteVisitors(retention);
-                    log += count.ToString() + " Visitors Purged<br />";
+                    try
+                    {
+                        count = visitorRepository.DeleteVisitors(retention);
+                        log += count.ToString() + " Visitors Purged<br />";
+                    }
+                    catch (Exception ex)
+                    {
+                        log += $"Error Purging Visitors - {ex.Message}<br />";
+                    }
                 }
             }