Merge pull request #136 from TaloDev/req-res-logging

Add request + response logging options

Author: tudddorrr

Assets/Talo Game Services/Talo/Runtime/APIs/BaseAPI.cs

@@ -67,6 +67,11 @@ protected async Task<string> Call(
 
             var allHeaders = continuity ? headers : BuildHeaders();
 
+            if (Talo.Settings.logRequests && Debug.isDebugBuild)
+            {
+                Debug.Log($"<-- {method} {uri}{(continuity ? " [CONTINUITY]" : "")} {content}");
+            }
+
             if (Talo.Settings.offlineMode)
             {
                 return HandleOfflineRequest(uri, method, content, allHeaders);
@@ -91,6 +96,11 @@ protected async Task<string> Call(
                     await Task.Yield();
                 }
 
+                if (Talo.Settings.logResponses && Debug.isDebugBuild)
+                {
+                    Debug.Log($"--> {method} {uri} [{www.responseCode}] {www.downloadHandler.text}");
+                }
+
                 if (www.result == UnityWebRequest.Result.Success)
                 {
                     return www.downloadHandler.text;

Assets/Talo Game Services/Talo/Runtime/SocketResponses/SocketResponse.cs

@@ -1,8 +1,9 @@
+using System;
 using UnityEngine;
 
 namespace TaloGameServices
 {
-    [System.Serializable]
+    [Serializable]
     public class SocketResponse
     {
         private class SocketMessage<T>
@@ -13,20 +14,28 @@ private class SocketMessage<T>
 
         public string message;
 
-        public SocketResponse(string message) 
+        public SocketResponse(string message)
         {
             this.message = message;
         }
 
         public string GetResponseType()
         {
-            var message = JsonUtility.FromJson<SocketMessage<object>>(this.message);
-            return message.res;
+            var json = JsonUtility.FromJson<SocketMessage<object>>(message);
+            return json.res;
         }
 
         public T GetData<T>()
         {
             return JsonUtility.FromJson<SocketMessage<T>>(message).data;
         }
+
+        public object GetJsonData()
+        {
+            var json = message.Substring(1, message.Length - 2) // remove the curly braces
+                .Replace("\"res\":\"" + GetResponseType() + "\",", "") // remove the response type
+                .Replace("\"data\":", ""); // remove the data key, only keep the value
+            return json;
+        }
     }
 }

Assets/Talo Game Services/Talo/Runtime/TaloSettings.cs

@@ -16,5 +16,9 @@ public class TaloSettings : ScriptableObject
         public bool offlineMode = false;
         [Tooltip("Whether to automatically connect to the Talo socket when the game starts")]
         public bool autoConnectSocket = true;
+        [Tooltip("Enable request logs in Debug builds")]
+        public bool logRequests = true;
+        [Tooltip("Enable response logs in Debug builds")]
+        public bool logResponses = true;
     }
 }

Assets/Talo Game Services/Talo/Runtime/TaloSocket.cs

@@ -43,6 +43,11 @@ private void HandleMessage(WebSocketConnection connection, WebSocketMessage wsm)
             var res = response.GetResponseType();
             OnMessageReceived?.Invoke(response);
 
+            if (Talo.Settings.logResponses && Debug.isDebugBuild)
+            {
+                Debug.Log($"--> WSS {res} {response.GetJsonData()}");
+            }
+
             switch (res)
             {
                 case "v1.connected":
@@ -83,7 +88,13 @@ public void CloseConnection()
 
         public void Send<T>(SocketRequest<T> request)
         {
-            socket.AddOutgoingMessage(JsonUtility.ToJson(request));
+            var data = JsonUtility.ToJson(request);
+
+            if (Talo.Settings.logRequests && Debug.isDebugBuild)
+            {
+                Debug.Log($"<-- WSS {request.req} {data}");
+            }
+            socket.AddOutgoingMessage(data);
         }
 
         private void IdentifyPlayer()