Matteo Collina
Matteo Collina
I'm not using this module.. if you'd like this fixed I'd love to review a PR.
isn't the snippet you pasted enough?
That's not supported.
Can you include what you are trying to achieve? The snippet is not enough.
Your usecase is a bit far from mine. Would you like to send a Pull Request to address this issue? Remember to add unit tests.
@rbadr can you simplify that repository by removing all traces of Nest? I'm not familiar with it and debugging something through all indirection is very hard. Could you reproduce just...
This is because `customProps` is called twice: 1. https://github.com/pinojs/pino-http/blob/3361ced517772b18087c49cb7f32b44e779179ce/logger.js#L107-L110 2. https://github.com/pinojs/pino-http/blob/3361ced517772b18087c49cb7f32b44e779179ce/logger.js#L144-L147 I think https://github.com/pinojs/pino-http/pull/197 had a bad bug we didn't see at the time. I think we should have have...
> @mcollina I don't see a way to build an artifact and test all changes in my local. No information in the README.md or ci.yml, please could you provide the...
We should really split `customProps` into two functions, not one.
Could you please provide a complete example to reproduce your problem?