chore: fix the timestamp in instana logger from epoch time to ISO string

packages/collector/src/logger.js

@@ -95,7 +95,9 @@ exports.init = function init(userConfig = {}) {
     parentLogger = uninstrumentedLogger({
       name: '@instana/collector',
       level: 'info',
-      base: { threadId, pid: process.pid, hostname: os.hostname() }
+      base: { threadId, pid: process.pid, hostname: os.hostname() },
+      // Set ISO timestamp format for the default logger
+      timestamp: () => `,"time":"${new Date().toISOString()}"`
     });
   }
 
@@ -121,7 +123,8 @@ exports.init = function init(userConfig = {}) {
       const pinoOpts = {
         ...parentLogger.levels,
         level: parentLogger.level || 'info',
-        base: parentLogger.bindings()
+        base: parentLogger.bindings(),
+        timestamp: () => `,"time":"${new Date().toISOString()}"`
       };
 
       // CASE: Either its the customers pino instance (and probably a different pino installation/version)
@@ -132,7 +135,8 @@ exports.init = function init(userConfig = {}) {
       const symbols = Object.getOwnPropertySymbols(instance);
 
       // CASE: We copy any settings from the pino instance such as custom formats.
-      if (symbols && Array.isArray(symbols)) {
+      if (userConfig.logger && symbols && Array.isArray(symbols)) {
+        // If there's a custom timestamp function from the userConfig logger  , we'll respect that
         const timeSym = symbols.find(sym => String(sym) === 'Symbol(pino.time)');
         // @ts-ignore
         const timestampFn = instance[timeSym];

packages/collector/test/logger_test.js

@@ -139,6 +139,29 @@ describe('logger', () => {
       stream => expect(stream.level).to.equal('info')
     );
   });
+
+  it('should use ISO timestamp format for pino logger', () => {
+    const logger = log.init({});
+
+    // Create a mock write stream to capture the log output
+    const capturedLogs = [];
+    const mockStream = {
+      write: function (chunk) {
+        capturedLogs.push(JSON.parse(chunk));
+      }
+    };
+
+    const originalStream = logger.logger[uninstrumentedLogger.symbols.streamSym];
+    logger.logger[uninstrumentedLogger.symbols.streamSym] = mockStream;
+
+    logger.info('Test log message');
+
+    logger.logger[uninstrumentedLogger.symbols.streamSym] = originalStream;
+
+    expect(capturedLogs.length).to.equal(1);
+    expect(capturedLogs[0]).to.have.property('time');
+    expect(capturedLogs[0].time).to.match(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/);
+  });
 });
 
 /**