Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support the new TraceId and SpanId in Serilog #220

Open
kimsagro1 opened this issue Jan 2, 2024 · 3 comments
Open

Support the new TraceId and SpanId in Serilog #220

kimsagro1 opened this issue Jan 2, 2024 · 3 comments

Comments

@kimsagro1
Copy link

Describe your suggestion
Serilog 3.1 adds first-class support for TraceId and SpanId:
serilog/serilog#1955

Each sink has to implement this.

@pinkfloydx33
Copy link

This would require upgrading to the latest version of Serilog. This package is using an outdated version which we, as consumers, would like to get upgraded as well

@pinkfloydx33
Copy link

10 months later; I guess this project is dead...?

@nblumhardt
Copy link
Contributor

Not dead, but needs contributions. It's a high-use package, so I guess the main thing holding up PRs for these kinds of things is the effort involved in testing and pulling something together at a suitable level of quality. A PR would be very welcome if anyone following this thread is keen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants