Adrian Cole
Adrian Cole
example which breaks on /async endpoint ```java @SpringBootApplication @Controller open class Demo @Autowired constructor(val ctx: CurrentTraceContext) { @GetMapping("/") fun sync(): ResponseEntity { return ResponseEntity.ok(ctx.get().traceIdString()); } @GetMapping("/async") fun async() = runBlocking...
@ackintosh thank you .. I started thinking this, too. the problem is that this assumes a thread local is behind the impl, which is almost always the case, but might...
@ackintosh do you know some way to globally register TracingContextElement so people don't need to add this to their code explicitly?
very nice link
I think this is the way to do it.. can you please verify? ```kotlin import brave.propagation.CurrentTraceContext import brave.propagation.CurrentTraceContext.Scope import brave.propagation.TraceContext import kotlinx.coroutines.ThreadContextElement import kotlin.coroutines.AbstractCoroutineContextElement import kotlin.coroutines.CoroutineContext class TracingContextElement( val currentTraceContext:...
@ackintosh can you use a different http client? I want to make sure there's no bug in the `WebClient` instrumentation which might be leaking its state. Not saying there is,...
ps use `(TracingContextElement(ctx)` not `(TracingContextElement(Tracing.current().currentTraceContext()))`, though this should have nothing to do with the problem
could possibly be a bug. worth trying to isolate into a non kotlin test in sleuth. I will be offline after tomorrow for holiday fyi this feature can go into...
@shiraji question: do you need to reference like `Tracing.current().currentTraceContext()` or is it just that you have no handy reference to CurrentTraceContext and doing so with the current tracing component is...
note actually the only OS dist I could get this to pass on was trusty. @vietj It is possible there's something subtle going on here. Is there anything native related...