Merge pull request #10 from podium-lib/error-listeners

Add error event listeners on all metric streams
podium-lib · Mar 5, 2019 · 2f8f623 · 2f8f623
2 parents 0d9d141 + d6f12d8
commit 2f8f623
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/lib/proxy.js b/lib/proxy.js
@@ -78,8 +78,12 @@ const PodiumProxy = class PodiumProxy {
             value: pathRegex(regExPath, this.pathnameEntries),
         });
 
+        this.metrics.on('error', error => {
+            this.log.error('Error emitted by metric stream in @podium/proxy module', error);
+        });
+
         this.proxy.on('error', error => {
-            this.log.error(error);
+            this.log.error('Error emitted by proxy in @podium/proxy module', error);
         });
 
         this.proxy.on('proxyReq', (proxyReq, req, res) => {

diff --git a/package.json b/package.json
@@ -40,14 +40,14 @@
         "ttl-mem-cache": "4.0.2"
     },
     "devDependencies": {
-        "eslint": "^5.14.1",
+        "eslint": "^5.15.0",
         "eslint-config-airbnb-base": "^13.1.0",
-        "eslint-config-prettier": "^4.0.0",
+        "eslint-config-prettier": "^4.1.0",
         "eslint-plugin-import": "^2.14.0",
         "eslint-plugin-prettier": "^3.0.0",
         "jest": "^24.1.0",
         "prettier": "^1.14.2",
-        "readable-stream": "^3.1.1"
+        "readable-stream": "^3.2.0"
     },
     "jest": {
         "coveragePathIgnorePatterns": [