@autofleet/rabbit 3.1.0 → 3.1.1

package/dist/index.js

@@ -128,15 +128,6 @@ class RabbitMq {
         const user = trace?.context?.get(consts_1.USER_OBJECT);
         const traceId = trace?.context?.get(consts_1.TRACING_HEADER);
         const outbreakTrace = zehut_1.outbreak.getCurrentContext();
-        const obj = Object.fromEntries(trace?.context);
-        const jsonString = JSON.stringify(obj);
-        logger_1.default.info('getPublishOptions', {
-            traceId,
-            user,
-            traceId2: outbreakTrace?.context?.get(consts_1.TRACING_HEADER),
-            traceContextSize: trace?.context?.size,
-            jsonString,
-        });
         return {
             timestamp: moment_1.default().unix(),
             headers: {
@@ -349,11 +340,13 @@ class RabbitMq {
                 }
                 const traceId = msg.properties.headers[consts_1.TRACING_HEADER];
                 const userId = msg.properties.headers[consts_1.USER_TRACING_HEADER];
-                logger_1.default.info('rabbit: consume', { traceId, userId, enableRabbitTrace });
                 const parsedMessage = RabbitMq.parseMsg(msg);
                 const releaseLock = await this.lockRedisIfNeeded(parsedMessage, optionsWithDefaults);
                 const trace = zehut_1.newTrace(zehut_1.traceTypes.RABBIT);
+                // setting also outbreak trace as part of legacy code
                 const outbreakTrace = zehut_1.outbreak.newTrace(zehut_1.traceTypes.RABBIT);
+                // enableRabbitTrace is a flag to protect from different flows that doesn't work with permission
+                // and we don't want to fail the flow because of it
                 if (userId && enableRabbitTrace) {
                     try {
                         await Promise.all([
@@ -366,24 +359,10 @@ class RabbitMq {
                         await this.nack(channel, queue, optionsWithDefaults, { messageTtl: deadMessageTtl }, msg, releaseLock)(msg);
                     }
                 }
-                logger_1.default.info('rabbit consume - first trace', {
-                    trace,
-                    outbreakTrace,
-                    user: trace?.context?.get(consts_1.USER_OBJECT),
-                });
                 if (traceId) {
                     trace?.context?.set(consts_1.TRACING_HEADER, traceId);
                     outbreakTrace?.context.set(consts_1.TRACING_HEADER, traceId);
-                    logger_1.default.info('rabbit consume - second trace', {
-                        trace,
-                        outbreakTrace,
-                        user: trace?.context?.get(consts_1.USER_OBJECT),
-                        traceId: trace?.context?.get(consts_1.TRACING_HEADER),
-                    });
                 }
-                logger_1.default.info('rabbit consume - third trace', {
-                    trace: zehut_1.getCurrentPayload(),
-                });
                 if (auditContext) {
                     await auditContext(queue);
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autofleet/rabbit",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "",
   "main": "dist/index.js",
   "scripts": {

package/src/index.ts

@@ -116,15 +116,6 @@ class RabbitMq implements IAfRabbitMq {
     const user = trace?.context?.get(USER_OBJECT);
     const traceId = trace?.context?.get(TRACING_HEADER);
     const outbreakTrace = outbreak.getCurrentContext();
-    const obj = Object.fromEntries(trace?.context);
-    const jsonString = JSON.stringify(obj);
-    logger.info('getPublishOptions', {
-      traceId,
-      user,
-      traceId2: outbreakTrace?.context?.get(TRACING_HEADER),
-      traceContextSize: trace?.context?.size,
-      jsonString,
-    });
     return {
       timestamp: moment().unix(),
       headers: {
@@ -493,11 +484,13 @@ class RabbitMq implements IAfRabbitMq {
 
           const traceId = msg.properties.headers[TRACING_HEADER];
           const userId = msg.properties.headers[USER_TRACING_HEADER];
-          logger.info('rabbit: consume', { traceId, userId, enableRabbitTrace });
           const parsedMessage = RabbitMq.parseMsg(msg);
           const releaseLock = await this.lockRedisIfNeeded(parsedMessage, optionsWithDefaults);
           const trace = newTrace(traceTypes.RABBIT);
+          // setting also outbreak trace as part of legacy code
           const outbreakTrace = outbreak.newTrace(traceTypes.RABBIT);
+          // enableRabbitTrace is a flag to protect from different flows that doesn't work with permission
+          // and we don't want to fail the flow because of it
           if (userId && enableRabbitTrace) {
             try {
               await Promise.all([
@@ -510,27 +503,11 @@ class RabbitMq implements IAfRabbitMq {
             }
           }
 
-          logger.info('rabbit consume - first trace', {
-            trace,
-            outbreakTrace,
-            user: trace?.context?.get(USER_OBJECT),
-          });
-
           if (traceId) {
             (trace as any)?.context?.set(TRACING_HEADER, traceId);
             (outbreakTrace as any)?.context.set(TRACING_HEADER, traceId);
-            logger.info('rabbit consume - second trace', {
-              trace,
-              outbreakTrace,
-              user: trace?.context?.get(USER_OBJECT),
-              traceId: trace?.context?.get(TRACING_HEADER),
-              // traceId2: outbreakTrace?.context?.get(TRACING_HEADER),
-            });
           }
 
-          logger.info('rabbit consume - third trace', {
-            trace: getCurrentPayload(),
-          });
           if (auditContext) {
             await auditContext(queue);
           }