@instana/core - npm Package Compare versions

Comparing version 4.5.0 to 4.5.1

CHANGELOG.md

@@ -6,2 +6,8 @@ # Change Log
 
+## [4.5.1](https://github.com/instana/nodejs/compare/v4.5.0...v4.5.1) (2025-01-13)
+
+### Bug Fixes
+
+- resolved bunyan npm installation warning ([#1447](https://github.com/instana/nodejs/issues/1447)) ([c4abc75](https://github.com/instana/nodejs/commit/c4abc75e8f0fb347310ce2bc6a0c24f8c3343b15))
+
 # [4.5.0](https://github.com/instana/nodejs/compare/v4.4.0...v4.5.0) (2024-12-16)
@@ -8,0 +14,0 @@

package.json

 {
   "name": "@instana/core",
-  "version": "4.5.0",
+  "version": "4.5.1",
   "description": "Core library for Instana's Node.js packages",
@@ -75,3 +75,3 @@ "main": "src/index.js",
   },
-  "gitHead": "eb9b10bc5328c03893f4948dfb717fa3982db278"
+  "gitHead": "41c09b8fdd23f6cb0df992c7e479d71532ac91b4"
 }

src/tracing/instrumentation/loggers/bunyan.js

@@ -32,6 +32,5 @@ /*
     function () {
-      if (arguments.length === 0 || (this.fields && !!this.fields.__in)) {
+      if (arguments.length === 0) {
         // * arguments.length === 0 -> This is a logger.warn() type of call (without arguments), this will not log
         // anything but simply return whether the log level in question is enabled for this logger.
-        // * this.fields.__in -> This is one of Instana's own loggers, we never want to trace those log calls.
         return originalLog.apply(this, arguments);
@@ -38,0 +37,0 @@ }

src/tracing/instrumentation/loggers/pino.js

@@ -21,2 +21,5 @@ /*
 exports.init = function init() {
+  // TODO: Fix the issue with Pino instrumentation. If Pino is required multiple times,
+  //       only the first instance gets instrumented. This behavior is caused by `onFileLoad`.
+  //       Fix is being tracked in https://jsw.ibm.com/browse/INSTA-23066.
   hook.onFileLoad(/\/pino\/lib\/tools\.js/, instrumentPinoTools);
@@ -23,0 +26,0 @@ };

src/util/initializedTooLateHeuristic.js

@@ -35,3 +35,3 @@ /*
   /\/aws-sdk\/lib\//,
-  // deliberately not including bunyan because we depend on bunyan ourselves
+  /\/bunyan\/lib\//,
   /\/bull\/index.js/,
@@ -66,3 +66,5 @@ /\/cls-hooked\/index.js/,
   /\/pg\/lib\//,
-  /\/pino\/lib\//,
+  // deliberately not including pino because we depend on pino as our default logger
+  // this is not a clean way of skipping it, but this is the workaround for now.
+  // investigate as part of https://jsw.ibm.com/browse/INSTA-22756
   /\/redis\/dist\/index.js/,
@@ -69,0 +71,0 @@ /\/redis\/index.js/,

No alert changes

Improved metrics

Total package byte prevSize

799043

increased by0.07%

Lines of code

20350

increased by0.02%

No dependency changes