`FST_ERR_REP_ALREADY_SENT` when wrapping route handler about fastify-static HOT 5 CLOSED

Does the proposed solution not show that?

Yes, consider the follow code working properly without @autotelic/fastify-opentelemetry.
You should not force everyone to use async or return Promise.

instance.get('/ok', function(request, reply) {
  setTimeout(function() {
    reply.send('ok')
  }, 1000)
})

The occur of FST_ERR_REP_ALREADY_SENT is you are forking the promise chain.
The plugin you mention actually doing the following action.

// note the async here
instance.get('/ok', async function(request, reply) {
  setTimeout(function() {
    reply.send('ok') // you return to client inside timeout as well
  }, 1000)
  // you implicit return undefined which reply to client with empty string
})

from fastify-static.

I am closing this issue because nothing show the bug is exist inside this repository.
Please minimize your reproducible snippet without any other plugins.

from fastify-static.

My suggestion is remove the async from @autotelic/fastify-opentelemetry.
So, it will respect the original function.

from fastify-static.

I am closing this issue because nothing show the bug is exist inside this repository. Please minimize your reproducible snippet without any other plugins.

Does the proposed solution not show that?

from fastify-static.

Thanks for your help @climba03003. I opened a PR to @autotelic/fastify-opentelemetry.

from fastify-static.