fix: resolved logging objects being undefined or missing

 - refs INSTA-12533
 - some log messages did not log the corresponding objects

packages/aws-fargate/src/activate.js

@@ -30,7 +30,9 @@ function init() {
 
   metrics.init(config, function onReady(err, ecsContainerPayload) {
     if (err) {
-      logger.error('Initializing @instana/aws-fargate failed. This fargate task will not be monitored.', err);
+      logger.error(
+        `Initializing @instana/aws-fargate failed. This fargate task will not be monitored. ${err.message} ${err.stack}`
+      );
       metrics.deactivate();
       return;
     }
@@ -68,7 +70,8 @@ function init() {
       // eslint-disable-next-line no-unused-expressions
       process.send && process.send('instana.aws-fargate.initialized');
     } catch (e) {
-      logger.error('Initializing @instana/aws-fargate failed. This fargate task will not be monitored.', e);
+      logger.error(`Initializing @instana/aws-fargate failed. This fargate task will not be monitored. 
+        ${JSON.stingify(e)}`);
     }
   });
 }

packages/aws-lambda/src/metrics/npmPackageDescription.js

@@ -16,7 +16,7 @@ exports.currentPayload = undefined;
 exports.activate = function activate() {
   coreUtil.applicationUnderMonitoring.getMainPackageJsonStartingAtDirectory(rootDir.root, (err, pckg) => {
     if (err) {
-      logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      logger.warn(`Failed to determine main package json. Reason: ${err.message} ${err.stack}`);
     }
 
     if (!err && pckg) {

packages/aws-lambda/src/metrics/npmPackageName.js

@@ -16,7 +16,7 @@ exports.currentPayload = undefined;
 exports.activate = function activate() {
   coreUtil.applicationUnderMonitoring.getMainPackageJsonStartingAtDirectory(rootDir.root, (err, pckg) => {
     if (err) {
-      logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      logger.warn(`Failed to determine main package json. Reason: ${err.message} ${err.stack}`);
     }
 
     if (!err && pckg) {

packages/aws-lambda/src/metrics/npmPackageVersion.js

@@ -16,7 +16,7 @@ exports.currentPayload = undefined;
 exports.activate = function activate() {
   coreUtil.applicationUnderMonitoring.getMainPackageJsonStartingAtDirectory(rootDir.root, (err, pckg) => {
     if (err) {
-      logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      logger.warn(`Failed to determine main package json. Reason: ${err.message} ${err.stack}`);
     }
 
     if (!err && pckg) {

packages/collector/src/actions/source.js

@@ -37,7 +37,7 @@ exports.getSourceFile = (request, multiCb) => {
 function readFile(request, multiCb) {
   fs.readFile(request.args.file, { encoding: 'utf8' }, (error, content) => {
     if (error) {
-      logger.debug('Failed to retrieve source file for user request: %s.', request.args.file, { error });
+      logger.debug(`Failed to retrieve source file for user request: ${request.args.file}. ${error}`);
       multiCb({
         error: `Could not load file. Error: ${error.message}`
       });

packages/collector/src/agent/requestHandler.js

@@ -61,9 +61,8 @@ function handleRequest(request) {
 function sendResponse(request, response) {
   agentConnection.sendAgentResponseToAgent(request.messageId, response, error => {
     if (error) {
-      logger.warn('Failed to send agent response for action %s and message ID %s', request.action, request.messageId, {
-        error
-      });
+      logger.warn(`Failed to send agent response for action ${request.action} and message ID ${request.messageId}`);
+      logger.warn(`Error: ${error}`);
     }
   });
 }

packages/collector/src/agentConnection.js

@@ -271,7 +271,7 @@ exports.sendSpans = function sendSpans(spans, cb) {
       logger.debug(`Failed to send: ${JSON.stringify(spanInfo)}`);
     } else {
       const spanInfo = getSpanLengthInfo(spans);
-      logger.debug(`Successfully sent:${JSON.stringify(spanInfo)}`);
+      logger.debug(`Successfully sent: ${JSON.stringify(spanInfo)}`);
     }
     cb(err);
   });
@@ -374,7 +374,7 @@ function sendData(path, data, cb, ignore404 = false) {
   if (typeof logger.trace === 'function') {
     logger.trace('Sending data to %s', path);
   } else {
-    logger.debug('Sending data to %s', path, agentOpts);
+    logger.debug('Sending data to %s %o', path, agentOpts);
   }
 
   // Convert payload to a buffer to correctly identify content-length ahead of time.

packages/collector/src/announceCycle/agentready.js

@@ -214,7 +214,9 @@ function sendEOLEvent() {
     },
     err => {
       if (err) {
-        logger.debug('Sending a monitoring event for the Node.js version end-of-life check has failed.', err);
+        logger.debug(
+          `Sending a monitoring event for the Node.js version end-of-life check has failed. ${JSON.stringify(err)}`
+        );
       }
     }
   );

packages/collector/src/announceCycle/unannounced.js

@@ -93,11 +93,9 @@ function tryToAnnounce(ctx, retryDelay = initialRetryDelay) {
       agentResponse = JSON.parse(rawResponse);
     } catch (e) {
       logger.error(
-        "Failed to parse the JSON payload from the Instana host agent's response. Establishing the " +
-          'connection to the Instana host agent will be retried in %s ms. The response payload was %s.',
-        retryDelay,
-        rawResponse,
-        e
+        "Failed to parse the JSON payload from the Instana host agent's response. Establishing the connection" +
+          `to the Instana host agent will be retried in ${retryDelay} ms. The response payload was ${rawResponse}.` +
+          `${e?.message} ${e?.stack}`
       );
       setTimeout(tryToAnnounce, retryDelay, ctx, nextRetryDelay).unref();
       return;

packages/collector/src/index.js

@@ -118,7 +118,7 @@ function init(_config) {
     require('./metrics').init(config);
   }
 
-  logger.info('@instana/collector module version:', require(path.join(__dirname, '..', 'package.json')).version);
+  logger.info('@instana/collector module version: %s', require(path.join(__dirname, '..', 'package.json')).version);
   require('./announceCycle').start();
 
   return init;

packages/core/src/tracing/cls.js

@@ -310,11 +310,11 @@ function putPseudoSpan(spanName, kind, traceId, spanId) {
   span.k = kind;
 
   if (!traceId) {
-    logger.warn('Cannot start a pseudo span without a trace ID', spanName, kind);
+    logger.warn(`Cannot start a pseudo span without a trace ID": ${spanName}, ${kind}`);
     return;
   }
   if (!spanId) {
-    logger.warn('Cannot start a pseudo span without a span ID', spanName, kind);
+    logger.warn(`Cannot start a pseudo span without a span ID": ${spanName}, ${kind}`);
     return;
   }
 

packages/core/src/tracing/spanBuffer.js

@@ -458,7 +458,7 @@ function transmitSpans() {
   // since sending spans downstream will take a few milliseconds, even that will be rare (and it is acceptable).
   downstreamConnection.sendSpans(spansToSend, function sendSpans(/** @type {Error} */ error) {
     if (error) {
-      logger.warn(`Failed to transmit spans, will retry in ${transmissionDelay} ms.`, error.message);
+      logger.warn(`Failed to transmit spans, will retry in ${transmissionDelay} ms. ${error?.message} ${error?.stack}`);
       spans = spans.concat(spansToSend);
       removeSpansIfNecessary();
     }

packages/core/src/util/applicationUnderMonitoring.js

@@ -98,7 +98,7 @@ function readFile(packageJsonPath, cb) {
     try {
       parsedMainPackageJson = JSON.parse(contents);
     } catch (e) {
-      logger.warn('Package.json file %s cannot be parsed: %s', packageJsonPath, e);
+      logger.warn(`Package.json file ${packageJsonPath} cannot be parsed: ${e?.message} ${e?.stack}`);
       return cb(e, null);
     }
     return cb(null, parsedMainPackageJson);

packages/core/src/util/atMostOnce.js

@@ -32,9 +32,10 @@ module.exports = function atMostOnce(name, cb) {
     if (callCount === 1) {
       return cb.apply(null, arguments);
     }
-
-    logger.debug('Function %s was called %s times. This time with the following arguments.', name, callCount, {
+    const argObj = {
       args: Array.prototype.slice.call(arguments)
-    });
+    };
+
+    logger.debug(`Function ${name} was called ${callCount} times. This time with the following arguments: ${argObj}`);
   };
 };

packages/google-cloud-run/src/activate.js

@@ -38,8 +38,7 @@ function init() {
   metrics.init(config, function onReady(err, serviceRevisionPayload) {
     if (err) {
       logger.error(
-        'Initializing @instana/google-cloud-run failed. This container instance will not be monitored.',
-        err
+        `Initializing @instana/google-cloud-run failed. This container instance will not be monitored. ${err}`
       );
       metrics.deactivate();
       return;

packages/serverless-collector/src/activate.js

@@ -47,7 +47,9 @@ async function init() {
     // eslint-disable-next-line no-unused-expressions
     process.send && process.send('instana.serverless-collector.initialized');
   } catch (e) {
-    logger.error('Initializing @instana/serverless-collector failed. This process will not be traced.', e);
+    logger.error(
+      `Initializing @instana/serverless-collector failed. This process will not be traced. ${e?.message} ${e?.stack}`
+    );
   }
 }
 

packages/serverless-collector/src/metrics/name.js

@@ -19,7 +19,7 @@ module.exports = (config, logger) => {
   return new Promise(resolve => {
     instanaCore.util.applicationUnderMonitoring.getMainPackageJsonStartingAtMainModule(config, (err, packageJson) => {
       if (err) {
-        logger.debug('Failed to determine main package.json.', err);
+        logger.debug(`Failed to determine main package.json.', ${JSON.stingify(err)}`);
         return resolve();
       }
 

packages/serverless/src/backend_connector.js

@@ -404,7 +404,10 @@ function send(resourcePath, payload, finalLambdaRequest, callback) {
             e
           );
         } else {
-          logger.warn('Could not send traces and metrics to Instana. The Instana back end seems to be unavailable.', e);
+          logger.warn(
+            `Could not send traces and metrics to Instana. The Instana back end seems to be unavailable. 
+            ${JSON.stringify(e)}`
+          );
         }
       }
 

packages/shared-metrics/src/description.js

@@ -31,7 +31,7 @@ exports.activate = function activate(config) {
   attempts++;
   applicationUnderMonitoring.getMainPackageJsonStartingAtMainModule(config, (err, packageJson) => {
     if (err) {
-      return logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      return logger.warn(`Failed to determine main package json. Reason: ${err.message} ${err.stack}`);
     } else if (!packageJson && attempts < MAX_ATTEMPTS) {
       setTimeout(() => {
         exports.activate(config);

packages/shared-metrics/src/healthchecks.js

@@ -79,7 +79,7 @@ function gatherHealthcheckResults() {
     .catch(function onHealthcheckResultFailure(/** @type {*} */ err) {
       // @ts-ignore
       exports.currentPayload = {};
-      logger.warn('Unexpected error while getting healthcheck results', err);
+      logger.warn(`Unexpected error while getting healthcheck results. ${err?.message} ${err?.stack}`);
       timeoutHandle = setTimeout(gatherHealthcheckResults, timeBetweenHealthcheckCalls);
       timeoutHandle.unref();
     });

packages/shared-metrics/src/keywords.js

@@ -31,7 +31,7 @@ exports.activate = function activate(config) {
   attempts++;
   applicationUnderMonitoring.getMainPackageJsonStartingAtMainModule(config, (err, packageJson) => {
     if (err) {
-      return logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      return logger.warn(`Failed to determine main package json. Reason: ${err?.message} ${err?.stack}`);
     } else if (!packageJson && attempts < MAX_ATTEMPTS) {
       setTimeout(() => {
         exports.activate(config);

packages/shared-metrics/src/name.js

@@ -32,7 +32,7 @@ exports.activate = function activate(config) {
 
   applicationUnderMonitoring.getMainPackageJsonStartingAtMainModule(config, (err, packageJson) => {
     if (err) {
-      return logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      return logger.warn(`Failed to determine main package json. Reason: ${err?.message} ${err?.stack}`);
     } else if (!packageJson && attempts < exports.MAX_ATTEMPTS) {
       logger.debug('Main package.json could not be found. Will try again later.');
 

packages/shared-metrics/src/util/nativeModuleRetry.js

@@ -206,12 +206,12 @@ function copyPrecompiled(opts, loaderEmitter, callback) {
             callback(true);
           })
           .catch(error => {
-            logger.warn(`Copying the precompiled build for ${opts.nativeModuleName} ${label} failed.`, error);
+            logger.warn(`Copying the precompiled build for ${opts.nativeModuleName} ${label} failed. ${error}`);
             callback(false);
           });
       })
       .catch(tarErr => {
-        logger.warn(`Unpacking the precompiled build for ${opts.nativeModuleName} ${label} failed.`, tarErr);
+        logger.warn(`Unpacking the precompiled build for ${opts.nativeModuleName} ${label} failed. ${tarErr}`);
         callback(false);
       });
   });
@@ -245,7 +245,9 @@ function findNativeModulePath(opts) {
     opts.nativeModuleParentPath = path.join(opts.nativeModulePath, '..');
     return true;
   } catch (e) {
-    logger.debug(`Could not find location for ${opts.nativeModuleName}. Will create a path for it.`, e);
+    logger.debug(
+      `Could not find location for ${opts.nativeModuleName}. Will create a path for it. ${e?.message} ${e.stack}`
+    );
     return createNativeModulePath(opts);
   }
 }

packages/shared-metrics/src/version.js

@@ -30,7 +30,7 @@ exports.activate = function activate(config) {
   attempts++;
   applicationUnderMonitoring.getMainPackageJsonStartingAtMainModule(config, (err, packageJson) => {
     if (err) {
-      return logger.warn('Failed to determine main package json. Reason: ', err.message, err.stack);
+      return logger.warn(`Failed to determine main package json. Reason: ${err?.message} ${err?.stack}`);
     } else if (!packageJson && attempts < MAX_ATTEMPTS) {
       setTimeout(() => {
         exports.activate(config);