fix issue with alert manager concurrency

RaidMax · May 15, 2023 · 4002745 · 4002745
1 parent 2af03d1
commit 4002745
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Application/Alerts/AlertManager.cs b/Application/Alerts/AlertManager.cs
@@ -55,7 +55,7 @@ public IEnumerable<Alert.AlertState> RetrieveAlerts(EFClient client)
                 alerts = alerts.Concat(_states[client.ClientId].AsReadOnly());
             }
 
-            return alerts.OrderByDescending(alert => alert.OccuredAt);
+            return alerts.OrderByDescending(alert => alert.OccuredAt).ToList();
         }
         finally
         {

diff --git a/SharedLibraryCore/BaseController.cs b/SharedLibraryCore/BaseController.cs
@@ -178,7 +178,7 @@ public override async void OnActionExecuting(ActionExecutingContext context)
             ViewBag.ReportCount = Manager.GetServers().Sum(server =>
                 server.Reports.Count(report => DateTime.UtcNow - report.ReportedOn <= TimeSpan.FromHours(24)));
             ViewBag.PermissionsSet = PermissionsSet;
-            ViewBag.Alerts = AlertManager.RetrieveAlerts(Client).ToList();
+            ViewBag.Alerts = AlertManager.RetrieveAlerts(Client);
 
             base.OnActionExecuting(context);
         }