Manuel Spigolon comments

Results 381 comments of


                                            Manuel Spigolon

docs: add Incident Response Process (IRP)

Closing as not strictly needed

`reply.send(string)` does not serialize strings as JSON even if the `Content-Type` is set to `application/json`

I checked the v4 and it has the same issue, but I think the doc is correct and the behaviour is wrong > I checked the relevant logic and it...

check named-hooks

closing in favour of: https://github.com/fastify/otel/issues/110

fix: ensure validation errors after async preValidation reach error handler (#6214)

The coverage seems not fine If we compare this PR with this other one: - https://github.com/fastify/fastify/pull/6247 This one seems a sligtly better because it adds a single try/catch execution compared...

fix: ensure validation errors after async preValidation reach error handler (#6214)

This code: https://github.com/fastify/fastify/pull/6229/files#diff-15a9e086692377c86b02752c54c928f4ca6f697758fc5dacc09bbde1d2dff96bR75-R76 is never reached due the new try/catch, so we need to remove it to restore the coverage

Upgrade or switch from glob

What if those deps save us from some injection? I say that because I trust the `glob`'s maintainer and his work. In this scenario, the number of dependancies is not...

FTP connection reset errors.

To fix it there is a workaround: https://github.com/SamKirkland/FTP-Deploy-Action/issues/489#issuecomment-2660873337

feat: export HTTP status codes types

Any news on this?

import/require fastify module is incredibly slow > 300 ms

@spaceemotion could you try? ``` const noOpLogic = function factoryNoOp() { return function compilerNoOp() { return function routeNoOp() { return true } } } const app = require('fastify')({ logger: true,...