pinojs / pino-http

🌲 high-speed HTTP logger for Node.js
MIT License
539 stars 117 forks source link

nestedKey now working properly #307

Open stychu opened 11 months ago

stychu commented 11 months ago

I have this pinohttp logger setup with nestedKey. The log is not respecting req property to go inside the nestedKey if defined.

export const httpLoggerMiddleware = pinoHttp({
  redact: ['req.headers', 'res.headers'],
  msgPrefix: '[HTTP] ',
  nestedKey: 'logPayload',
});
  req: {
      "id": 4,
      "method": "GET",
      "url": "/asdsadad",
      "query": {
        "batch": "1",
      },
      "params": {},
      "headers": "[Redacted]",
      "remoteAddress": "::1",
      "remotePort": 52680
    }
    logPayload: {
      "res": {
        "statusCode": 200,
        "headers": "[Redacted]"
      },
      "responseTime": 15
    }
mcollina commented 11 months ago

This looks like a bug! Would you like to send a PR?