getsentry / getsentry/sentry-dotnet
DataCollection phase 2: migrate OpenTelemetry and logging integrations
- Dominant language
- C#
- Stars
- 770
- Forks
- 248
- Avg merge
- 2d 22h
- Merged PRs (30d)
- 51
Description
Part of #5420. Phase 2 — consumer migration.
Migrate remaining integrations to the resolved config:
- `src/Sentry.OpenTelemetry/SentrySpanProcessor.cs` — db/messaging attribute passthrough (`db.statement`-equivalents → `DatabaseQueryData`; messaging attrs → `Queues` where applicable).
- `src/Sentry.Serilog/SentrySinkExtensions.cs:20,255` — `sendDefaultPii` sink parameter (keep mapping to the legacy option until the major; document).
- `src/Sentry.NLog/SentryTarget.cs:490-502` — user layouts → these are user-*configured* (explicit), so per spec they are NOT gated; confirm and add a test locking that in.
- `src/Sentry.Extensions.Logging` — anything reading `SendDefaultPii` via options binding.
Contributor guide
Research direction
Start with src/Sentry.OpenTelemetry/SentrySpanProcessor.cs, src/Sentry.Serilog/SentrySinkExtensions.cs, src/Sentry.NLog/SentryTarget.cs, and the SendDefaultPii handling in src/Sentry.Extensions.Logging. Trace each integration's resolved-config and options-binding paths, then verify database and messaging attributes, the Serilog legacy mapping, and the NLog user-layout behavior. Done means the remaining integrations follow the phase 2 specification and the NLog behavior has a locking test.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- observability
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100