From 5dea025553ad8249b09ad3decf6e3c4d62d1c074 Mon Sep 17 00:00:00 2001
From: Adam McCrea <adam@adamlogic.com>
Date: Thu, 6 Jun 2024 10:37:12 -0400
Subject: [PATCH] fix: Don't include the entire config object in reports

This could include complex objects like Redis connections. We only need a few pieces of information, so let's be selective about them.
---
 packages/node-core/src/report.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/node-core/src/report.js b/packages/node-core/src/report.js
index 265da8a..b5346dc 100644
--- a/packages/node-core/src/report.js
+++ b/packages/node-core/src/report.js
@@ -14,9 +14,12 @@ class Report {
     return {
       container: this.config.container,
       pid: process.pid,
-      // Convert logger (DerivedLogger instance) into something sane
-      config: { ...this.config, logger: this.config.logger && this.config.logger.constructor.name },
       adapters: adapterMetadata,
+      config: {
+        version: this.config.version,
+        container: this.config.container,
+        log_level: this.config.log_level,
+      },
       metrics: this.metrics.map((metric) => [
         metric.time.getTime() / 1000,
         metric.value,