@opentelemetry/instrumentation-redis - npm Package Compare versions

Comparing version 0.28.0 to 0.29.0

build/src/utils.js

@@ -73,3 +73,3 @@ "use strict";
             attributes: {
-                [semantic_conventions_1.SemanticAttributes.DB_SYSTEM]: _1.RedisInstrumentation.COMPONENT,
+                [semantic_conventions_1.SemanticAttributes.DB_SYSTEM]: semantic_conventions_1.DbSystemValues.REDIS,
                 [semantic_conventions_1.SemanticAttributes.DB_STATEMENT]: dbStatementSerializer(cmd.command, cmd.args),
@@ -86,3 +86,3 @@ },
         if (this.address) {
-            span.setAttribute(semantic_conventions_1.SemanticAttributes.NET_PEER_IP, `redis://${this.address}`);
+            span.setAttribute(semantic_conventions_1.SemanticAttributes.DB_CONNECTION_STRING, `redis://${this.address}`);
         }
@@ -98,3 +98,5 @@ const originalCallback = arguments[0].callback;
                     }, err => {
-                        api_1.diag.error('Error executing responseHook', err);
+                        if (err) {
+                            api_1.diag.error('Error executing responseHook', err);
+                        }
                     }, true);
@@ -101,0 +103,0 @@ }

build/src/version.d.ts

@@ -1,2 +0,2 @@
-export declare const VERSION = "0.28.0";
+export declare const VERSION = "0.29.0";
 //# sourceMappingURL=version.d.ts.map

build/src/version.js

@@ -20,3 +20,3 @@ "use strict";
 // this is autogenerated file, see scripts/version-update.js
-exports.VERSION = '0.28.0';
+exports.VERSION = '0.29.0';
 //# sourceMappingURL=version.js.map

CHANGELOG.md

 # Changelog
 
+## [0.29.0](https://github.com/open-telemetry/opentelemetry-js-contrib/compare/instrumentation-redis-v0.28.0...instrumentation-redis-v0.29.0) (2022-03-02)
+
+
+### ⚠ BREAKING CHANGES
+
+* **mysql*,redis:** net.peer.ip -> db.connection_string
+
+### Bug Fixes
+
+* **mysql*,redis:** net.peer.ip -> db.connection_string ([bf39b90](https://github.com/open-telemetry/opentelemetry-js-contrib/commit/bf39b908fd64cec92c862b2deca5d760ddcf4509))
+* **opentelemetry-instrumentation-redis:** add condition before error print ([#897](https://github.com/open-telemetry/opentelemetry-js-contrib/issues/897)) ([f1d2fd0](https://github.com/open-telemetry/opentelemetry-js-contrib/commit/f1d2fd084c8e3e494e3606c4eca53158495f43f6))
+
 ## [0.28.0](https://www.github.com/open-telemetry/opentelemetry-js-contrib/compare/instrumentation-redis-v0.27.0...instrumentation-redis-v0.28.0) (2022-01-24)
@@ -51,2 +63,2 @@
   * devDependencies
-    * @opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0
+    * @opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

package.json

 {
   "name": "@opentelemetry/instrumentation-redis",
-  "version": "0.28.0",
+  "version": "0.29.0",
   "description": "OpenTelemetry redis automatic instrumentation package.",
@@ -22,3 +22,2 @@ "main": "build/src/index.js",
     "compile": "npm run version:update && tsc -p .",
-    "codecov": "nyc report --reporter=json && codecov -f coverage/*.json -p ../../",
     "prepare": "npm run compile"
@@ -60,4 +59,3 @@ },
     "@types/mocha": "7.0.2",
-    "@types/node": "14.17.9",
-    "codecov": "3.8.3",
+    "@types/node": "16.11.21",
     "cross-env": "7.0.3",
@@ -78,3 +76,3 @@ "gts": "3.1.0",
   },
-  "gitHead": "edfc04cbe3e7faaa365724ddc26bf702b68cc008"
+  "gitHead": "78717b6dda677bf737811c34df21ef0ec88c31a3"
 }

README.md

@@ -23,3 +23,3 @@ # OpenTelemetry redis Instrumentation for Node.js
 
-- `>=2.6.0`
+- `^2.6.0 || ^3.0.0` (version `4` is not yet supported)
 
@@ -26,0 +26,0 @@ ## Usage

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

42221

increased by1.82%

Dev dependency count

16

decreased by-5.88%

Lines of code

337

increased by0.6%

No dependency changes