getsentry / getsentry/sentry-dotnet

DataCollection phase 2: migrate OpenTelemetry and logging integrations

Open
#5,435 1 comment 0 reactions 0 assignees View on GitHub
.NET Feature
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.