fix: update serializeLogData function implementation

Expensify · Sep 4, 2024 · 876adf8 · 876adf8
1 parent bf9b00e
commit 876adf8
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/libs/Middleware/Logging.ts b/src/libs/Middleware/Logging.ts
@@ -25,8 +25,13 @@ function getCircularReplacer() {
     };
 }
 
-function serializeLoggingData(logData?: Record<string, unknown> | null): Record<string, unknown> | undefined | null {
-    return logData && (JSON.parse(JSON.stringify(logData, getCircularReplacer())) as Record<string, unknown>);
+function serializeLoggingData<T extends Record<string, unknown> | undefined>(logData: T): T | null {
+    try {
+        return JSON.parse(JSON.stringify(logData, getCircularReplacer())) as T;
+    } catch (error) {
+        Log.hmmm('Failed to serialize log data', {error});
+        return null;
+    }
 }
 
 function logRequestDetails(message: string, request: Request, response?: Response | void) {