Skip to content

Support correlating logs with traces with Timbre OTLP appender#399

Closed
devurandom wants to merge 1 commit intotaoensso:masterfrom
devurandom:ds/fix-f3ce2b5c3f4c1daa0462f611c87097c56fe2d6e0
Closed

Support correlating logs with traces with Timbre OTLP appender#399
devurandom wants to merge 1 commit intotaoensso:masterfrom
devurandom:ds/fix-f3ce2b5c3f4c1daa0462f611c87097c56fe2d6e0

Conversation

@devurandom
Copy link
Contributor

Capture the OpenTelemetry context in a Timbre middleware and pick it
up in the Timbre OTLP appender. This allows to correlate logs with
traces.

Fixes: f3ce2b5

Capture the OpenTelemetry context in a Timbre middleware and pick it
up in the Timbre OTLP appender.  This allows to correlate logs with
traces.

Fixes: f3ce2b5
@devurandom devurandom changed the title Support passing OpenTelemetry context to Timbre OTLP appender Support correlating logs with traces with Timbre OTLP appender Aug 21, 2025
@ptaoussanis ptaoussanis self-assigned this Aug 21, 2025
@devurandom devurandom marked this pull request as ready for review August 21, 2025 10:43
ptaoussanis pushed a commit that referenced this pull request Aug 21, 2025
…ndom)

This allows the correlation of logs and traces.
Follows f3ce2b5
@ptaoussanis
Copy link
Member

@devurandom Thanks for this Dennis!

I made a few changes at 080acff and have pushed for testing at https://clojars.org/com.taoensso/timbre/versions/6.8.0-SNAPSHOT

Main change was to rename the context key to make it easier for folks to work with, if they choose to.

If you have an opportunity, could you please confirm that this works as expected? If so, I'll aim to cut a new Timbre release later today or tomorrow 👍

@devurandom
Copy link
Contributor Author

Yes, 6.8.0-SNAPSHOT works. Thanks!

@devurandom devurandom closed this Aug 21, 2025
@devurandom devurandom deleted the ds/fix-f3ce2b5c3f4c1daa0462f611c87097c56fe2d6e0 branch August 21, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants