express-pino-logger icon indicating copy to clipboard operation
express-pino-logger copied to clipboard

Incorrect readme.md example

Open vjpr opened this issue 2 years ago • 1 comments

https://github.com/pinojs/express-pino-logger#custom-serializers

'use strict'

var app = require('express')()
+ var ExpressPinoLogger = require('express-pino-logger')
- var ExpressPinoLogger = require('express-pino-logger')() <-------- remove the `()` because we pass `serializers` below.

var pino = ExpressPinoLogger({
  serializers: {
    req: (req) => ({
      method: req.method,
      url: req.url,
      user: req.raw.user,
    }),
  },
})

// middleware that augments the req - must be added before the pino middleware
app.use((req, res, next) => {
  req.user = 'testing';
  next();
})

app.use(pino)

vjpr avatar Aug 25 '21 21:08 vjpr

Good spot! Would you like to send a Pull Request to address this issue?

mcollina avatar Aug 26 '21 07:08 mcollina