Merge pull request #54 from Moesif/express-minor-fix

dgilling · web-flow · commit b7918feaba0f · 2021-09-07T19:58:01.000-07:00
default to localhost if hostname not found
diff --git a/lib/formatEventDataAndSave.js b/lib/formatEventDataAndSave.js
@@ -85,7 +85,7 @@ function safeGetHostname(req) {
   try {
     return req.hostname;
   } catch(err) {
-    return req.headers && req.headers['x-forwarded-host'];
+    return (req.headers && req.headers['x-forwarded-host']) || 'localhost';
   }
 }
 
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "moesif-nodejs",
-  "version": "3.1.11",
+  "version": "3.1.12",
   "description": "Monitoring agent to log API calls to Moesif for deep API analytics",
   "main": "lib/index.js",
   "keywords": [

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ function safeGetHostname(req) {`
`85`	`85`	`try {`
`86`	`86`	`return req.hostname;`
`87`	`87`	`} catch(err) {`
`88`		`- return req.headers && req.headers['x-forwarded-host'];`
	`88`	`+ return (req.headers && req.headers['x-forwarded-host']) \|\| 'localhost';`
`89`	`89`	`}`
`90`	`90`	`}`
`91`	`91`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "moesif-nodejs",`
`3`		`- "version": "3.1.11",`
	`3`	`+ "version": "3.1.12",`
`4`	`4`	`"description": "Monitoring agent to log API calls to Moesif for deep API analytics",`
`5`	`5`	`"main": "lib/index.js",`
`6`	`6`	`"keywords": [`