replyWithErrorPage TypeScript types break when Fastify is initiated with http2: true

[punkpeye]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/node

SDK Version

8.30

Framework Version

Fastify@4.28.1

Link to Sentry event

No response

Reproduction Example/SDK Setup

No response

Steps to Reproduce

const app = fastify({
  bodyLimit: 32 * 1_024 * 1_024,
  http2: true,
  trustProxy: true,
});


app.setErrorHandler((error, request, reply) => {
  return replyWithErrorPage(reply, error);
});

Expected Result

No TypeScript error.

Actual Result

Argument of type 'FastifyReply<Http2Server, Http2ServerRequest, Http2ServerResponse, RouteGenericInterface, unknown, FastifySchema, FastifyTypeProviderDefault, unknown>' is not assignable to parameter of type 'FastifyReply<RawServerDefault, IncomingMessage, ServerResponse<IncomingMessage>, RouteGenericInterface, unknown, FastifySchema, FastifyTypeProviderDefault, unknown>'.
  Type 'Http2ServerRequest' is missing the following properties from type 'IncomingMessage': headersDistinct, trailersDistinctts(2345)
(parameter) reply: FastifyReply<Http2Server, Http2ServerRequest, Http2ServerResponse, RouteGenericInterface, unknown, FastifySchema, FastifyTypeProviderDefault, unknown>

[mydea]

Thank you for raising this! We probably need to widen the return type here - I'll put this in our backlog to work on, PRs for this are also welcome :)

[mydea]

Hey, revisiting this - where exactly is Sentry in this? I just checked again, and the snippet you provide does not use any Sentry code.