quarkus
quarkus copied to clipboard
Add traceresponse header to HTTP responses
https://github.com/quarkusio/quarkus/issues/11931
/cc @brunobat (opentelemetry), @radcortez (opentelemetry)
Should it be done in dev mode only ?
@wottop Are you thinking on continuing the work on this issue? Some considerations:
- The response header should not be inserted by default. A new property is needed with default value to false.
- This needs to be developed for quarkus-rest quarkus-resteasy-classic and vertx routes.
- We need tests.
@brunobat What should we do here?
Like it is, we cannot accept the PR and because @wottop hasn't addressed the comments I'm closing the PR. We can re-open it if we hear any news.
I tried that code in one of my projects. Unfortunately it also won't address resources retiring Multi responses (see also #44898). So I guess there is some alternative implementation needed to fully support setting the traceresponse.