James Sumners
James Sumners
That is my opinion, yes.
> Because no package manager does anything with the field except to write a warning log to stdout. The result is people filing issues about the warning instead of paying...
https://github.com/fastify/fastify/pull/5667#issuecomment-2435514579 and https://github.com/fastify/fastify/pull/5667#issuecomment-2435548304 say exactly what I originally wrote, and re-posted in https://github.com/fastify/fastify/pull/5667#issuecomment-2435726746 > Your position @jsumners seems to be that you don't want to add this unless the package...
The team should start an issue like the Express one that has linked to this PR. If the team decides that the field should be maintained, then it should be...
Sorry, it's not clear to me what you are asking for here. Can you provide some code and output (current and desired) samples?
Would you like to send a Pull Request to address this issue?
I really don't have any input to provide here. I wouldn't use AWS Lambda for anything that would require any Fastify branded module. Lambda is a tool for background batch...
Would you like to send a Pull Request to address this issue? Remember to add unit tests.
> This changeset only modifies the following behviour: neither logged _Request_ nor _Response_ objects are not transformed by their corresponding `stdSerializers` unconditionally, at the start of `LOG` ([lib/tools.js](https://github.com/pinojs/pino/pull/2081/files#diff-7f88797c7db38a6af7a3283e3af30f2cdd3c2489bd9c85a7a25b32ec2485f724L43)). > >...
> Yes, this is correct. When custom serializers _are_ provided and they depend on the changes made by the standard serializers on the target objects (ie: to _sanitize_ them?), they...