Matteo Collina

Results 3427 comments of Matteo Collina

Take a look at https://github.com/pinojs/pino-http/issues/350

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.

I'm mildly concerned on the long term maintainability of this, as it essentially rewrites the module - backporting _might_ lead to more churn than needed. You might get less churn...

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Thanks for reporting. It would be great to add this feature somehow to fastify-swagger. I'm moving this issue there.

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Can someone add a reproduction for this?

There actuall was a memory leak caused by `withFilter` not calling the `.return()` method of the internal filter: https://github.com/mercurius-js/mercurius/pull/1170.