From ff58b499d94d7ac1d6c2bba706661b82dbc80ffe Mon Sep 17 00:00:00 2001
From: Nikos Kostoulas <nikostoulas@users.noreply.github.com>
Date: Thu, 5 Oct 2023 11:34:26 +0300
Subject: [PATCH] Add prefix to inbound header logs (#97)

- Updated the mapInRes function in transformers.js to include headers in metaHeaders
- Adjusted tests in requestProxyTest.js to reflect the changes in transformers.js
- This change will help in better log management by differentiating inbound headers
---
 lib/transformers/transformers.js              |  3 +--
 test/lib/adapters/default/requestProxyTest.js | 12 ++++--------
 2 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/lib/transformers/transformers.js b/lib/transformers/transformers.js
index 18f9bff..14f4328 100644
--- a/lib/transformers/transformers.js
+++ b/lib/transformers/transformers.js
@@ -104,8 +104,7 @@ const mapInRes = (res, req, startedAt, reqId, opts) => {
     contentLength,
     userAgent,
     log_tag: logTags.CATEGORY.INBOUND_RESPONSE.TAG,
-    metaHeaders: { request: req.metaHeaders },
-    headers
+    metaHeaders: { request: req.metaHeaders, ...headers }
   };
 };
 
diff --git a/test/lib/adapters/default/requestProxyTest.js b/test/lib/adapters/default/requestProxyTest.js
index 57ad793..aa98f7b 100644
--- a/test/lib/adapters/default/requestProxyTest.js
+++ b/test/lib/adapters/default/requestProxyTest.js
@@ -30,7 +30,8 @@ describe('#defaultAdapter', () => {
         opts: {
           request: {
             enabled: true
-          }
+          },
+          headersRegex: new RegExp('^X-.*', 'i')
         }
       });
       const options = {
@@ -59,7 +60,7 @@ describe('#defaultAdapter', () => {
       const httpRequest = () => {
         return {
           on: (on, cb) => {
-            const res = {};
+            const res = { headers: { 'x-foo': 'foo' } };
             cb(res);
             logger.info.callCount.should.equal(2);
             logger.info.args[0][0].should.eql({
@@ -89,8 +90,7 @@ describe('#defaultAdapter', () => {
               contentLength: 0,
               userAgent: '',
               log_tag: 'inbound_response',
-              headers: {},
-              metaHeaders: { request: {} }
+              metaHeaders: { request: {}, headers: { 'x-foo': 'foo' } }
             });
           }
         };
@@ -276,7 +276,6 @@ describe('#defaultAdapter', () => {
         contentLength: 0,
         userAgent: '',
         log_tag: 'inbound_response',
-        headers: {},
         metaHeaders: { request: {} }
       });
     });
@@ -417,7 +416,6 @@ describe('#defaultAdapter', () => {
         contentLength: 0,
         userAgent: '',
         log_tag: 'inbound_response',
-        headers: {},
         metaHeaders: { request: {} }
       });
     });
@@ -491,7 +489,6 @@ describe('#defaultAdapter', () => {
         contentLength: 0,
         userAgent: '',
         log_tag: 'inbound_response',
-        headers: {},
         metaHeaders: {
           request: {}
         }
@@ -718,7 +715,6 @@ describe('#defaultAdapter', () => {
               contentLength: 0,
               userAgent: '',
               log_tag: 'inbound_response',
-              headers: {},
               metaHeaders: {
                 request: {}
               }