Add missing route config type

[FabianFrank]

as of fastify 4.19.0 route configs have to be typed explicitly, without this change with fastify 4.19.0 and above a route like

f.get("/foo", { config: { rawBody: true } }, () => {})

produces the following error:

Type '{ auth: "public"; rawBody: true; }' is not assignable to type 'Omit<FastifyContextConfig & FastifyRouteConfig, "url" | "method">'.
  Object literal may only specify known properties, and 'rawBody' does not exist in type 'Omit<FastifyContextConfig & FastifyRouteConfig, "url" | "method">'.

See fastify/fastify-rate-limit#306 for a similar fix in an official fastify plugin.

[Eomm]

Thanks for the PR!

Could you update the test/types/types.test.ts file too?

[FabianFrank]

Could you update the test/types/types.test.ts file too?

done

[Eomm]

Could you update the test/types/types.test.ts file too?

done

Sorry, I pushed a line because the test was passing without the plugin.d.ts changes

Landing now!

[kperdomo1]

hey! just wanted to comment that for servers with fastify versions lower than v4.19.0 this conflicts with other plugins typing, since before that this wasn't required before then 😢

for example, my server also imports '@fastify/rate-limit' with fastify v4.7.0. For this version, such explicit typing wasn't required. And because fastify-raw-body does so it completely overwrites the config context, so only rawBody option is accepted

maybe add fastify v4.19.0+ as peer dependency?

[Eomm]

why updating a plugin and not the server?
I mean, there are some security releases too since 4.7.0

Anyway, we should bump the version here:

https://github.com/Eomm/fastify-raw-body/blob/main/plugin.js#L106

[kperdomo1]

Lack of time :) Our server is fairly big and we'd need to jump a few minor versions from fastify. With that some other plugins as well and sadly we don't have very good testing in place.

Anyways, thanks for the response! Happy to open the PR