From d1f8e65a31d4d519622c64a7dfb6965ff37c6ae7 Mon Sep 17 00:00:00 2001
From: Antoine Arlaud <antoine@snyk.io>
Date: Wed, 18 Dec 2024 17:29:50 +0100
Subject: [PATCH] chore: disable body logging

---
 lib/client/config/configHelpers.ts    | 2 +-
 lib/client/index.ts                   | 2 ++
 test/functional/server-client.test.ts | 1 +
 3 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/lib/client/config/configHelpers.ts b/lib/client/config/configHelpers.ts
index 88291346c..37ca393c1 100644
--- a/lib/client/config/configHelpers.ts
+++ b/lib/client/config/configHelpers.ts
@@ -31,7 +31,7 @@ export const getClientConfigMetadata = (
     version: `${version}`,
     haMode: highAvailabilityModeEnabled(clientConfig),
     debugMode: clientConfig.logLevel === 'debug' ? true : false,
-    bodyLogMode: clientConfig.logEnableBody ? true : false,
+    bodyLogMode: clientConfig.logEnableBody === 'true' ? true : false,
     credPooling: isCredPoolingUsed(clientConfig),
     privateCa: clientConfig.nodeExtraCaCert ? true : false,
     tlsReject:
diff --git a/lib/client/index.ts b/lib/client/index.ts
index 66f7aed7f..fa777864b 100644
--- a/lib/client/index.ts
+++ b/lib/client/index.ts
@@ -53,6 +53,8 @@ export const main = async (clientOpts: ClientOpts) => {
     logger.info({ version }, 'Broker starting in client mode');
     let hookResults: HookResults = {};
     clientOpts.config.brokerClientId = uuidv4();
+    clientOpts.config.logEnableBody = 'false';
+    clientOpts.config.LOG_ENABLE_BODY = 'false';
     logger.info(
       { brokerClientId: clientOpts.config.brokerClientId },
       'generated broker client id',
diff --git a/test/functional/server-client.test.ts b/test/functional/server-client.test.ts
index 5e3025360..418390544 100644
--- a/test/functional/server-client.test.ts
+++ b/test/functional/server-client.test.ts
@@ -32,6 +32,7 @@ describe('proxy requests originating from behind the broker server', () => {
     process.env.BROKER_SERVER_URL = `http://localhost:${PORT}`;
     process.env.JIRAUSER = 'user';
     process.env.PASS = 'pass';
+    process.env.LOG_ENABLE_BODY = 'true'; // setting this to true to assert it connects with it disabled
 
     tws = await createTestWebServer();