pino-http - npm Package Compare versions

Comparing version 1.1.0 to 1.1.1

logger.js

@@ -38,3 +38,3 @@ 'use strict'
         err: err,
-        responseTime: this.responseTime
+        responseTime: responseTime
       }, 'request errored')
@@ -41,0 +41,0 @@ return

package.json

 {
   "name": "pino-http",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "High-speed HTTP logger for Node.js",
@@ -5,0 +5,0 @@ "main": "logger.js",

test.js

@@ -112,3 +112,2 @@ 'use strict'
 
-  var server = http.createServer(handle)
   function handle (req, res) {
@@ -121,10 +120,25 @@ logger(req, res)
 
-  t.tearDown(function (cb) {
-    server.close(cb)
-  })
+  expectResponseTime(t, dest, logger, handle)
+})
 
-  server.listen(0, '127.0.0.1', function (err) {
+test('responseTime for errored request', function (t) {
+  var dest = split(JSON.parse)
+  var logger = pinoLogger(dest)
+
+  function handle (req, res) {
+    logger(req, res)
+    setTimeout(function () {
+      res.emit('error', new Error('Some error'))
+      res.end()
+    }, 100)
+  }
+
+  expectResponseTime(t, dest, logger, handle)
+})
+
+function expectResponseTime (t, dest, logger, handle) {
+  setup(t, logger, function (err, server) {
     t.error(err)
     doGet(server)
-  })
+  }, handle)
 
@@ -136,3 +150,3 @@ dest.on('data', function (line) {
   })
-})
+}
 

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

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

11693

increased by2.99%

Lines of code

255

increased by4.51%

Number of medium supply chain risk alerts

9

decreased by-10%

No dependency changes