Refactors RequestLogs property to use ConcurrentBag for improved thread safety (#1333)

Author: waldekmastykarz

DevProxy/Logging/RequestLogger.cs

@@ -24,7 +24,7 @@ public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Except
         {
             if (_proxyState.IsRecording)
             {
-                _proxyState.RequestLogs.Add(requestLog);
+                _proxyState.RequestLogs.Enqueue(requestLog);
             }
 
             var requestLogArgs = new RequestLogArgs(requestLog);

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; }
+    ConcurrentQueue<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 ConcurrentQueue<RequestLog> RequestLogs { get; set; } = [];
     public Dictionary<string, object> GlobalData { get; set; } = new() {
         { ProxyUtils.ReportsKey, new Dictionary<string, object>() }
     };