fix: serialize request.data in logging middleware

src/libs/Middleware/Logging.ts

@@ -6,6 +6,25 @@ import type Request from '@src/types/onyx/Request';
 import type Response from '@src/types/onyx/Response';
 import type Middleware from './types';
 
+function getCircularReplacer() {
+    const ancestors: unknown[] = [];
+    return function (this: unknown, key: string, value: unknown): unknown {
+        if (typeof value !== 'object' || value === null) {
+            return value;
+        }
+        // `this` is the object that value is contained in, i.e the direct parent
+        // eslint-disable-next-line no-invalid-this
+        while (ancestors.length > 0 && ancestors.at(-1) !== this) {
+            ancestors.pop();
+        }
+        if (ancestors.includes(value)) {
+            return '[Circular]';
+        }
+        ancestors.push(value);
+        return value;
+    };
+}
+
 function logRequestDetails(message: string, request: Request, response?: Response | void) {
     // Don't log about log or else we'd cause an infinite loop
     if (request.command === 'Log') {
@@ -38,7 +57,10 @@ function logRequestDetails(message: string, request: Request, response?: Respons
      * requests because they contain sensitive information.
      */
     if (request.command !== 'AuthenticatePusher') {
-        extraData.request = request;
+        extraData.request = {
+            ...request,
+            data: JSON.parse(JSON.stringify(request.data, getCircularReplacer())) as Record<string, unknown>,
+        };
         extraData.response = response;
     }
 
@@ -125,3 +147,5 @@ const Logging: Middleware = (response, request) => {
 };
 
 export default Logging;
+
+export {getCircularReplacer};

tests/unit/LoggingMiddlewareTest.ts

@@ -0,0 +1,31 @@
+import {getCircularReplacer} from '@libs/Middleware/Logging';
+
+describe('LoggingMiddleware', () => {
+    describe('getCircularReplacer', () => {
+        it('should return "[Circular]" for circular references', () => {
+            const circularReplacer = getCircularReplacer();
+            const obj: Record<string, unknown> = {};
+            obj.obj = obj;
+            const result = JSON.stringify(obj, circularReplacer);
+            expect(result).toBe('{"obj":"[Circular]"}');
+        });
+
+        it('should return the original value for non-circular references', () => {
+            const circularReplacer = getCircularReplacer();
+            const obj: Record<string, unknown> = {};
+            obj.foo = 'bar';
+            const result = JSON.stringify(obj, circularReplacer);
+            expect(result).toBe('{"foo":"bar"}');
+        });
+
+        it('should not stringify function in the object', () => {
+            const circularReplacer = getCircularReplacer();
+            const obj: Record<string, unknown> = {
+                foo: () => 'bar',
+                baz: 'baz',
+            };
+            const result = JSON.stringify(obj, circularReplacer);
+            expect(result).toBe('{"baz":"baz"}');
+        });
+    });
+});