fix: Handle exceptions in callbacks gracefully

Author: kirre-bylund

Runtime/Client/LootLockerHTTPClient.cs

@@ -566,8 +566,10 @@ private void CallListenersAndMarkDone(LootLockerHTTPExecutionQueueItem execution
             executionItem.Done = true;
             response.requestContext = new LootLockerRequestContext(executionItem.RequestData.ForPlayerWithUlid, executionItem.RequestData.RequestStartTime);
             executionItem.Response = response;
+            if(!CompletedRequestIDs.Contains(executionItem.RequestData.RequestId)) {
+                CompletedRequestIDs.Add(executionItem.RequestData.RequestId);
+            }
             executionItem.RequestData.CallListenersWithResult(response);
-            CompletedRequestIDs.Add(executionItem.RequestData.RequestId);
         }
 
         private IEnumerator RefreshSession(string refreshForPlayerUlid, string forExecutionItemId, Action<LootLockerSessionResponse, string, string> onSessionRefreshedCallback)

Runtime/Client/LootLockerHttpRequestData.cs

@@ -87,7 +87,14 @@ public void CallListenersWithResult(LootLockerResponse response)
         {
             foreach(var listener in Listeners)
             {
-                listener?.Invoke(response);
+                try
+                {
+                    listener?.Invoke(response);
+                }
+                catch (Exception e)
+                {
+                    LootLockerLogger.Log($"Exception thrown in HTTP request listener for request id {RequestId}. Exception was: {e}.", LootLockerLogger.LogLevel.Error);
+                }
             }
             HaveListenersBeenInvoked = true;
         }