Bug: `customErrorMessage` actually got passed 4 arguments, but it's ts declaration only has 3 arguments

[gavinliukaiber]

customErrorMessage actually got passed 4 arguments:

pino-http/logger.js

Line 123 in 8baaeec

errorMessage(req, res, error, responseTime)

, but its ts declaration only has 3 arguments:

pino-http/index.d.ts

Line 34 in 8baaeec

customErrorMessage?: ((req: IM, res: SR, error: Error) => string) | undefined;

I'd like to make use of the 4th argument "responseTime" in my logging message.

[mcollina]

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests. We use tsd for testing our types

[vikasrathod4299]

I'd like to take this up. working on a PR will open it shortly!