@restorecommerce/kafka-client 0.4.1 → 0.4.4

package/CHANGELOG.md

@@ -3,6 +3,33 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [0.4.4](https://github.com/restorecommerce/kafka-client/compare/@restorecommerce/kafka-client@0.4.3...@restorecommerce/kafka-client@0.4.4) (2022-08-10)
+
+**Note:** Version bump only for package @restorecommerce/kafka-client
+
+
+
+
+
+## [0.4.3](https://github.com/restorecommerce/kafka-client/compare/@restorecommerce/kafka-client@0.4.2...@restorecommerce/kafka-client@0.4.3) (2022-07-07)
+
+**Note:** Version bump only for package @restorecommerce/kafka-client
+
+
+
+
+
+## [0.4.2](https://github.com/restorecommerce/kafka-client/compare/@restorecommerce/kafka-client@0.4.1...@restorecommerce/kafka-client@0.4.2) (2022-06-28)
+
+
+### Bug Fixes
+
+* **kafka-client:** to print error message and stack instead of entire error object ([a7ee20f](https://github.com/restorecommerce/kafka-client/commit/a7ee20f77a464589b5935ccd780f74b96dbc68a1))
+
+
+
+
+
 ## [0.4.1](https://github.com/restorecommerce/kafka-client/compare/@restorecommerce/kafka-client@0.4.0...@restorecommerce/kafka-client@0.4.1) (2022-06-10)
 
 

package/lib/events/provider/kafka/index.js

@@ -76,7 +76,7 @@ class Topic {
                             this.provider.logger.info(`Topic ${this.name} created successfully`);
                             resolve();
                         }).catch(err => {
-                            this.provider.logger.error(`Cannot create topic ${this.name}:`, err);
+                            this.provider.logger.error(`Cannot create topic ${this.name}:`, { code: err.code, message: err.message, stack: err.stack });
                             reject(err);
                         });
                     }
@@ -159,14 +159,14 @@ class Topic {
                     return this.provider.admin.fetchTopicOffsets(this.name).then(data => {
                         resolve(parseInt(data[0].offset, 10));
                     }).catch(err => {
-                        this.provider.logger.error('Error occurred retrieving topic offset:', err);
+                        this.provider.logger.error('Error occurred retrieving topic offset:', { code: err.code, message: err.message, stack: err.stack });
                         reject(err);
                     });
                 }
                 return this.provider.admin.fetchTopicOffsetsByTimestamp(this.name, time).then(data => {
                     resolve(parseInt(data[0].offset, 10));
                 }).catch(err => {
-                    this.provider.logger.error('Error occurred retrieving topic offset:', err);
+                    this.provider.logger.error('Error occurred retrieving topic offset:', { code: err.code, message: err.message, stack: err.stack });
                     reject(err);
                 });
             });
@@ -251,7 +251,7 @@ class Topic {
                     this.provider.logger.info(`Consumer disconnected from topic ${this.name}`);
                     this.consumer = undefined;
                 }).catch((err) => {
-                    this.provider.logger.error(`Error occurred unsubscribing ${eventName} on topic ${this.name}`, err);
+                    this.provider.logger.error(`Error occurred unsubscribing ${eventName} on topic ${this.name}`, { code: err.code, message: err.message, stack: err.stack });
                 });
             }
         });
@@ -275,7 +275,7 @@ class Topic {
             }
             catch (error) {
                 // do not commit offset
-                logger.error(`topic ${context.topic} error`, error);
+                logger.error(`topic ${context.topic} error`, { code: error.code, message: error.message, stack: error.stack });
                 throw error;
             }
         }
@@ -308,14 +308,14 @@ class Topic {
                 yield this.consumer.connect().then(() => {
                     this.provider.logger.info(`Consumer for topic '${this.name}' connected`);
                 }).catch(err => {
-                    this.provider.logger.error(`Consumer for topic '${this.name}' connection error: ${err}`);
+                    this.provider.logger.error(`Consumer for topic '${this.name}' connection error`, { code: err.code, message: err.message, stack: err.stack });
                 });
                 yield this.consumer.subscribe({
                     topic: this.name
                 }).then(() => {
                     this.provider.logger.info(`Consumer for topic '${this.name}' subscribed`);
                 }).catch(err => {
-                    this.provider.logger.error(`Consumer for topic '${this.name}' subscriber error: ${err}`);
+                    this.provider.logger.error(`Consumer for topic '${this.name}' subscriber error`, { code: err.code, message: err.message, stack: err.stack });
                 });
                 // On receiving the message on Kafka consumer put the message to async Queue.
                 if (queue) {
@@ -332,7 +332,7 @@ class Topic {
                         }
                     })
                 }).catch(err => {
-                    this.provider.logger.error(`Consumer for topic '${this.name}' failed to run: ${err}`);
+                    this.provider.logger.error(`Consumer for topic '${this.name}' failed to run`, { code: err.code, message: err.message, stack: err.stack });
                 });
                 this.consumer.seek({
                     topic: this.name,
@@ -408,7 +408,7 @@ class Topic {
                         partition: 0 // ?
                     }
                 ]).then(resolve).catch(err => {
-                    this.provider.logger.error('Error committing offset:', err);
+                    this.provider.logger.error('Error committing offset', { code: err.code, message: err.message, stack: err.stack });
                     reject(err);
                 });
             });
@@ -601,7 +601,7 @@ class Kafka {
             decodedMsg = MessageClass.decode(msg);
         }
         catch (err) {
-            this.logger.error(`error on decoding message with event ${eventName}:`, { message: msg, error: err });
+            this.logger.error(`error on decoding message with event ${eventName}`, { message: msg, errorCode: err.code, errorMessage: err.message, errorStack: err.stack });
         }
         return decodedMsg;
     }
@@ -651,13 +651,13 @@ class Kafka {
                         }
                         resolve(data);
                     }).catch((err) => {
-                        this.logger.error(`error sending event ${eventName} to topic ${topicName}`, err);
+                        this.logger.error(`error sending event ${eventName} to topic ${topicName}`, { code: err.code, message: err.message, stack: err.stack });
                         reject(err);
                     });
                 });
             }
             catch (err) {
-                this.logger.error(`error on sending event ${eventName} to topic ${topicName}`, err);
+                this.logger.error(`error on sending event ${eventName} to topic ${topicName}`, { code: err.code, message: err.message, stack: err.stack });
                 throw err;
             }
         });

package/lib/events/provider/local/index.js

@@ -83,7 +83,7 @@ class Topic {
             return new Promise((resolve, reject) => {
                 root.load(protoFilePath, { keepCase: true }, (err, root) => {
                     if (err) {
-                        that.logger.error('Error loading protobuf definition', err);
+                        that.logger.error('Error loading protobuf definition', { message: err.message, stack: err.stack });
                         reject(err);
                     }
                     resolve(root);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@restorecommerce/kafka-client",
-  "version": "0.4.1",
+  "version": "0.4.4",
   "description": "Opinionated Kafka Client for Microservices",
   "main": "./lib/index.js",
   "author": "n-fuse GmbH",
@@ -17,36 +17,36 @@
   ],
   "typings": "lib/index.d.ts",
   "dependencies": {
-    "@restorecommerce/logger": "^0.13.0",
-    "async": "^3.2.0",
-    "cls-rtracer": "^2.5.1",
+    "@restorecommerce/logger": "^0.13.1",
+    "async": "^3.2.4",
+    "cls-rtracer": "^2.6.2",
     "events": "^3.3.0",
-    "kafkajs": "^2.0.0",
+    "kafkajs": "^2.1.0",
     "lodash": "^4.17.21",
-    "protobufjs": "^6.11.2",
+    "protobufjs": "^6.11.3",
     "retry": "^0.13.1",
-    "winston": "^3.7.2"
+    "winston": "^3.8.1"
   },
   "devDependencies": {
-    "@types/async": "^3.2.13",
+    "@types/async": "^3.2.15",
     "@types/long": "^4.0.2",
     "@types/mocha": "^9.1.1",
-    "@types/node": "^17.0.33",
-    "@typescript-eslint/eslint-plugin": "^5.23.0",
-    "@typescript-eslint/eslint-plugin-tslint": "^5.23.0",
-    "@typescript-eslint/parser": "^5.23.0",
+    "@types/node": "^18.0.3",
+    "@typescript-eslint/eslint-plugin": "^5.30.5",
+    "@typescript-eslint/eslint-plugin-tslint": "^5.30.5",
+    "@typescript-eslint/parser": "^5.30.5",
     "coveralls": "^3.1.1",
     "cross-env": "^7.0.3",
-    "eslint": "^8.15.0",
+    "eslint": "^8.19.0",
     "eslint-plugin-prefer-arrow-functions": "^3.1.4",
     "mocha": "^10.0.0",
     "npm-run-all": "^4.1.5",
     "nyc": "^15.1.0",
     "rimraf": "^3.0.0",
     "should": "^13.2.3",
-    "ts-node": "^10.7.0",
+    "ts-node": "^10.8.2",
     "tslint": "^6.1.3",
-    "typescript": "^4.6.4"
+    "typescript": "^4.7.4"
   },
   "scripts": {
     "pretest": "npm run build && node setupTopics.js com.example.test",
@@ -62,5 +62,14 @@
   "engines": {
     "node": ">= 12.0.0"
   },
-  "gitHead": "2d453885b1c82df3e6aabbfbfe3b5a9c0b240277"
+  "nx": {
+    "targets": {
+      "build": {
+        "outputs": [
+          "./lib"
+        ]
+      }
+    }
+  },
+  "gitHead": "d9c96be84651883aa79c607df9431c758d59a40c"
 }