Refactors RequestLogs property to use ConcurrentBag for improved thread safety

Author: waldekmastykarz

DevProxy/Proxy/IProxyState.cs

@@ -3,7 +3,7 @@
 // See the LICENSE file in the project root for more information.
 
 using DevProxy.Abstractions.Proxy;
-using System.Collections.ObjectModel;
+using System.Collections.Concurrent;
 
 namespace DevProxy.Proxy;
 
@@ -13,5 +13,5 @@ public interface IProxyState
 {
     Dictionary<string, object> GlobalData { get; }
     bool IsRecording { get; set; }
-    Collection<RequestLog> RequestLogs { get; }
+    ConcurrentBag<RequestLog> RequestLogs { get; }
 }

DevProxy/Proxy/ProxyState.cs

@@ -4,14 +4,14 @@
 
 using DevProxy.Abstractions.Proxy;
 using DevProxy.Abstractions.Utils;
-using System.Collections.ObjectModel;
+using System.Collections.Concurrent;
 
 namespace DevProxy.Proxy;
 
 sealed class ProxyState : IProxyState
 {
     public bool IsRecording { get; set; }
-    public Collection<RequestLog> RequestLogs { get; set; } = [];
+    public ConcurrentBag<RequestLog> RequestLogs { get; set; } = [];
     public Dictionary<string, object> GlobalData { get; set; } = new() {
         { ProxyUtils.ReportsKey, new Dictionary<string, object>() }
     };