[backend] Reactivate logging limit (#9554)

Author: richard-julien

opencti-platform/opencti-graphql/src/config/conf.js

@@ -96,6 +96,7 @@ nconf.file('default', resolveEnvFile('default'));
 const appLogLevel = nconf.get('app:app_logs:logs_level');
 const appLogFileTransport = booleanConf('app:app_logs:logs_files', true);
 const appLogConsoleTransport = booleanConf('app:app_logs:logs_console', true);
+export const appLogLevelMaxDepthSize = nconf.get('app:app_logs:max_depth_size') ?? 5;
 export const appLogLevelMaxArraySize = nconf.get('app:app_logs:max_array_size') ?? 50;
 export const appLogLevelMaxStringSize = nconf.get('app:app_logs:max_string_size') ?? 5000;
 export const appLogExtendedErrors = booleanConf('app:app_logs:extended_error_message', false);
@@ -105,29 +106,31 @@ export const extendedErrors = (metaExtension) => {
   }
   return {};
 };
-export const limitMetaErrorComplexity = (obj) => {
-  if (Array.isArray(obj)) {
-    // Create a new array with a limited size
-    const limitedArray = obj.slice(0, appLogLevelMaxArraySize);
-    // Recursively process each item in the truncated array
-    const processedArray = [];
-    for (let i = 0; i < limitedArray.length; i += 1) {
-      processedArray[i] = limitMetaErrorComplexity(limitedArray[i]);
+export const limitMetaErrorComplexity = (obj, current_depth = 0) => {
+  if (obj !== null && current_depth > appLogLevelMaxDepthSize && typeof obj !== 'function') {
+    if (Array.isArray(obj)) {
+      // Create a new array with a limited size
+      const limitedArray = obj.slice(0, appLogLevelMaxArraySize);
+      // Recursively process each item in the truncated array
+      const processedArray = [];
+      for (let i = 0; i < limitedArray.length; i += 1) {
+        processedArray[i] = limitMetaErrorComplexity(limitedArray[i], current_depth + 1);
+      }
+      return processedArray;
     }
-    return processedArray;
-  }
-  if (typeof obj === 'string' && obj.length > appLogLevelMaxStringSize) {
-    return `${obj.substring(0, appLogLevelMaxStringSize - 3)}...`;
-  }
-  if (obj !== null && typeof obj === 'object') {
-    // Create a new object to hold the processed properties
-    const limitedObject = {};
-    const keys = Object.keys(obj); // Get the keys of the object
-    for (let i = 0; i < keys.length; i += 1) {
-      const key = keys[i];
-      limitedObject[key] = limitMetaErrorComplexity(obj[key]);
+    if (typeof obj === 'string' && obj.length > appLogLevelMaxStringSize) {
+      return `${obj.substring(0, appLogLevelMaxStringSize - 3)}...`;
+    }
+    if (typeof obj === 'object') {
+      // Create a new object to hold the processed properties
+      const limitedObject = {};
+      const keys = Object.keys(obj); // Get the keys of the object
+      for (let i = 0; i < keys.length; i += 1) {
+        const key = keys[i];
+        limitedObject[key] = limitMetaErrorComplexity(obj[key], current_depth + 1);
+      }
+      return limitedObject;
     }
-    return limitedObject;
   }
   return obj;
 };
@@ -253,8 +256,8 @@ export const logApp = {
     if (appLogTransports.length > 0 && appLogger.isLevelEnabled(level)) {
       const data = addBasicMetaInformation(LOG_APP, error, { ...meta, source: 'backend' });
       // Prevent meta information to be too massive.
-      // const limitedData = limitMetaErrorComplexity(data);
-      appLogger.log(level, message, data);
+      const limitedData = limitMetaErrorComplexity(data);
+      appLogger.log(level, message, limitedData);
     }
   },
   _logWithError: (level, messageOrError, meta = {}) => {