Delegate dates logging to process manager in prod

Author: Ndpnt

scripts/dataset/logger/index.js

@@ -10,7 +10,9 @@ logger.format = combine(
   printf(({ level, message, counter, hash, timestamp }) => {
     const prefix = counter && hash ? `${counter.toString().padEnd(6)} ${hash.padEnd(40)}` : '';
 
-    return `${timestamp} ${level.padEnd(15)} ${prefix.padEnd(50)} ${message}`;
+    const timestampPrefix = process.env.NODE_ENV !== 'production' ? `${timestamp} ` : '';
+
+    return `${timestampPrefix}${level.padEnd(15)} ${prefix.padEnd(50)} ${message}`;
   }),
 );
 

src/collection-api/logger.js

@@ -31,7 +31,11 @@ const logger = winston.createLogger({
   format: combine(
     colorize(),
     timestamp({ format: 'YYYY-MM-DD HH:mm:ss' }),
-    printf(({ level, message, timestamp }) => `${timestamp} ${level.padEnd(15)} ${message}`),
+    printf(({ level, message, timestamp }) => {
+      const timestampPrefix = process.env.NODE_ENV !== 'production' ? `${timestamp} ` : '';
+
+      return `${timestampPrefix}${level.padEnd(15)} ${message}`;
+    }),
   ),
   transports,
   rejectionHandlers: transports,

src/logger/index.js

@@ -10,21 +10,15 @@ const alignedWithColorsAndTime = combine(
   colorize(),
   timestamp({ format: 'YYYY-MM-DD HH:mm:ss' }),
   printf(({ level, message, timestamp, serviceId, termsType, documentId }) => {
-    let prefix = '';
+    const servicePrefix = serviceId && termsType
+      ? `${serviceId} — ${termsType}${documentId ? `:${documentId}` : ''}`
+      : '';
 
-    if (serviceId && termsType) {
-      prefix = `${serviceId} — ${termsType}`;
-    }
-
-    if (documentId) {
-      prefix = `${prefix}:${documentId}`;
-    }
+    const truncatedPrefix = servicePrefix.length > 75 ? `${servicePrefix.slice(0, 74)}…` : servicePrefix;
 
-    if (prefix.length > 75) {
-      prefix = `${prefix.substring(0, 74)}…`;
-    }
+    const timestampPrefix = process.env.NODE_ENV !== 'production' ? `${timestamp} ` : '';
 
-    return `${timestamp} ${level.padEnd(15)} ${prefix.padEnd(75)} ${message}`;
+    return `${timestampPrefix}${level.padEnd(15)} ${truncatedPrefix.padEnd(75)} ${message}`;
   }),
 );