Add timeouts

[Eomm]

demo/src/server.ts

Line 39 in 341d36e

const app = Fastify({

We should suggest to set the timeouts

Reference

• draft blog post: feat: how to set good timeouts Eomm/blog-posts#46

[jean-michelet]

Ok, you can push a PR if you want or I'll take a look at your post in the week.

[jean-michelet]

Great article.

There is a Typescript issue, there is no http.headersTimeout on ServerOptions:
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/459d0edabf1a20c16c53c96a6609c3400fe480e2/types/node/http.d.ts#L234

I think we need to push a PR on definitely typed.

[jean-michelet]

Possible workaround is:

({
  serverFactory: (requestListener) => {
    const server = http.createServer(requestListener)
    server.headersTimeout = 30_000

    return server
  }
}) satisfies FastifyHttpOptions<http.Server>

[Eomm]

There is a Typescript issue, there is no http.headersTimeout on ServerOptions:

Yeah, it seems so

For reference, the option has been added in node v18:

• https://nodejs.org/api/http.html#httpcreateserveroptions-requestlistener

[jean-michelet]

I've asked on Microsoft TypeScript discord and we need to push a PR, I'll do this tomorrow.
I will use the workaround if too long to be merged.