logfmt - npm Package Compare versions

Comparing version 0.17.0 to 0.18.0

lib/request_logger.js

@@ -11,6 +11,7 @@ var commonFormatter = function(req, res){
 
-  var ip = req.ip || req.header('x-forwarded-for')
+  var httpHeader = req.header && req.header('x-forwarded-for')
+  var ip = req.ip || httpHeader
                   || req.connection.remoteAddress;
 
-  return {
+  var requestData =  {
     ip: ip,
@@ -21,5 +22,9 @@ time: (new Date()).toISOString(),
     "status": res.statusCode,
-    content_length: res.get('content-length'),
-    content_type: res.get('content-type'),
   }
+
+  if(res.get){
+    requestData.content_length = res.get('content-length');
+    requestData.content_type = res.get('content-type');
+  }
+  return requestData;
 }
@@ -26,0 +31,0 @@

package.json

 {
   "name": "logfmt",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "description": "key=value logger and parser",
@@ -5,0 +5,0 @@ "main": "logfmt.js",

README.md

@@ -83,3 +83,3 @@ # node-logfmt
 
-### `logfmt.time([label], [data], callback(logger))`
+### `logfmt.time([label], [data], [callback(logger)])`
 
@@ -103,4 +103,11 @@ #### `logger.log([data], [stream])`
 
-String `label` changes the key `<string>=<milliseconds>ms`
+If you don't pass in callback you get the logger returned.
+```javascript
+var logger = logfmt.time();
+logger.log();
+//=> elapsed=1ms
+```
 
+String `label` changes the key to `<string>=<milliseconds>ms`
+
 ```javascript
@@ -107,0 +114,0 @@ logfmt.time('time', function(logger){

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

54587

increased by1.09%

Number of package files

44

increased by2.33%

Lines of code

1276

increased by1.11%

Number of lines in readme file

375

increased by1.9%

Worsened metrics

Number of medium supply chain risk alerts

7

increased by16.67%

No dependency changes