zalando / logbook / 5676

97%

master: 100%

push

github

MDC Context Propagation Fix for logbook-ktor Client (#2319)

* Release 4.0.4

* Development 4.1.0-SNAPSHOT

* MDC Context Propagation Fix for logbook-ktor Client

Response logging in LogbookClient happens in a launched coroutine that doesn't inherit MDC context, breaking traceId and other observability data.

The suggested fix changes `scope.launch { }`` → `scope.launch(coroutineContext) { }``

Also added `.minusKey(Job)` during context propagation to:
   - Keep: MDCContext, CoroutineDispatcher, CoroutineName, custom elements
   - Remove: Job (prevents the launched coroutine from having conflicting parents)

* remove MDC context cleanup duplication

780 of 812 branches covered (96.06%)

1 of 1 new or added line in 1 file covered. (100.0%)

3945 of 4073 relevant lines covered (96.86%)

0.97 hits per line