[Bugfix] incompatible opentelemetry package version (#138)

* Update dependencies

* Fix service name retrieval in `_addOpentelemetryContext`

* Update README.md

* Update test workflow dependencies

* Drop support for `node14` + test `node22`

* Update README.md

* close loggers in tests

* Update README.md

* Update package.json

Author: yotamloe

.github/workflows/test-nodejs.yml

@@ -11,15 +11,14 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [14.x, 16.x, 18.x, 20.x]
+        node-version: [ 16.x, 18.x, 20.x, 22.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v2
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
-          cache: 'npm'
       - run: npm ci
       - run: npm install --save request
       - run: npm run test

README.md

@@ -124,6 +124,12 @@ var logger = require('logzio-nodejs').createLogger({
   ```
 
 ## Update log
+**2.4.0**
+- Update dependencies:
+  - `@opentelemetry/context-async-hooks` -> `^2.0.0`
+  - `@opentelemetry/sdk-trace-node` -> `^2.0.0`
+- Fix service name retrieval in `_addOpentelemetryContext`
+- Drop support for `Node 14` 
 
 **2.3.0**
 - Add a method to flush the list of logs (@MarceloRGonc)

lib/logzio-nodejs.js

@@ -253,7 +253,7 @@ class LogzioLogger {
       if (span) {
         msg.trace_id = span.spanContext().traceId;
         msg.span_id = span.spanContext().spanId;
-        msg.service_name = span.resource._attributes['service.name'];
+        msg.service_name = span.resource.attributes['service.name'];
       }
     }
     log(msg, obj) {

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "logzio-nodejs",
   "description": "A nodejs implementation for sending logs to Logz.IO cloud service Copy of logzio-nodejs",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "author": "Gilly Barr <gilly@logz.io>",
   "maintainers": [
     {
@@ -48,8 +48,8 @@
   ],
   "dependencies": {
     "@opentelemetry/api": "^1.9.0",
-    "@opentelemetry/context-async-hooks": "^1.30.0",
-    "@opentelemetry/sdk-trace-node": "^1.30.0",
+    "@opentelemetry/context-async-hooks": "^2.0.0",
+    "@opentelemetry/sdk-trace-node": "^2.0.0",
     "axios": "^1.6.4",
     "json-stringify-safe": "5.0.1",
     "lodash.assign": "4.2.0",

package.json

@@ -61,6 +61,7 @@ describe('logger', () => {
         const loggedMessage = logger._createBulk.getCall(0).args[0][0];
         assert(loggedMessage.trace_id, 'trace_id should exist');
         assert(loggedMessage.span_id, 'span_id should exist');
+        logger.close();
       });
 
       it('should not attach traceId or spanId when no span is active', () => {
@@ -77,6 +78,7 @@ describe('logger', () => {
         const loggedMessage = logger._createBulk.getCall(0).args[0][0];
         assert.strictEqual(loggedMessage.trace_id, undefined, 'trace_id should not exist');
         assert.strictEqual(loggedMessage.span_id, undefined, 'span_id should not exist');
+        logger.close();
       });
     });
     describe('logs a single line', () => {