jaeger-client - npm Package Compare versions

Comparing version 3.17.0 to 3.17.1

CHANGELOG.md

 # Changes by Version
 
+## 3.17.1 (2019-10-22)
+
+* [bug fix] Do not apply adaptive sampler to child spans (#410) -- Yuri Shkuro
+* Add toString to reporters (#403) -- Andrea Di Giorgi
+
 ## 3.17.0 (2019-09-23)
@@ -4,0 +9,0 @@

dist/src/reporters/composite_reporter.js

@@ -45,2 +45,9 @@ 'use strict';
   }, {
+    key: 'toString',
+    value: function toString() {
+      return this.name() + '(' + this._reporters.map(function (reporter) {
+        return reporter.toString();
+      }).join(',') + ')';
+    }
+  }, {
     key: 'report',
@@ -47,0 +54,0 @@ value: function report(span) {

dist/src/reporters/in_memory_reporter.js

@@ -45,2 +45,7 @@ 'use strict';
   }, {
+    key: 'toString',
+    value: function toString() {
+      return this.name();
+    }
+  }, {
     key: 'report',
@@ -47,0 +52,0 @@ value: function report(span) {

dist/src/reporters/logging_reporter.js

@@ -50,2 +50,7 @@ 'use strict';
   }, {
+    key: 'toString',
+    value: function toString() {
+      return this.name();
+    }
+  }, {
     key: 'close',
@@ -52,0 +57,0 @@ value: function close(callback) {

dist/src/reporters/noop_reporter.js

@@ -39,2 +39,7 @@ 'use strict';
   }, {
+    key: 'toString',
+    value: function toString() {
+      return this.name();
+    }
+  }, {
     key: 'report',
@@ -41,0 +46,0 @@ value: function report(span) {}

dist/src/reporters/remote_reporter.js

@@ -78,2 +78,7 @@ 'use strict';
   }, {
+    key: 'toString',
+    value: function toString() {
+      return this.name();
+    }
+  }, {
     key: 'report',
@@ -80,0 +85,0 @@ value: function report(span) {

dist/src/samplers/per_operation_sampler.js

@@ -112,4 +112,8 @@ 'use strict';
       var outTags = {};
-      var isSampled = this.isSampled(span.operationName, outTags);
-      // NB: return retryable=true here since we can change decision after setOperationName().
+      var isSampled = false;
+      if (span.context()._samplingState.isLocalRootSpan(span.context())) {
+        isSampled = this.isSampled(span.operationName, outTags);
+      }
+      // returning retryable=true since we can change the sampling decision
+      // after the first call to setOperationName()
       return { sample: isSampled, retryable: true, tags: outTags };
@@ -121,3 +125,6 @@ }
       var outTags = {};
-      var isSampled = this.isSampled(span.operationName, outTags);
+      var isSampled = false;
+      if (span.context()._samplingState.isLocalRootSpan(span.context())) {
+        isSampled = this.isSampled(span.operationName, outTags);
+      }
       return { sample: isSampled, retryable: false, tags: outTags };
@@ -124,0 +131,0 @@ }

dist/src/version.js

@@ -13,2 +13,2 @@ // Copyright (c) 2019 Uber Technologies, Inc.
 
-module.exports = '3.17.0';
+module.exports = '3.17.1';

package.json

 {
   "name": "jaeger-client",
-  "version": "3.17.0",
+  "version": "3.17.1",
   "description": "Jaeger binding for OpenTracing Node",
@@ -5,0 +5,0 @@ "engines": {

New alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 24 instances in 1 package

Fixed alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

618047

increased by0.36%

Lines of code

5536

increased by0.62%

Number of low supply chain risk alerts

29

decreased by-47.27%

No dependency changes