getsentry / getsentry/sentry-dotnet

DataCollection phase 2: migrate Sentry.AspNetCore

Open
#5,430 1 comment 0 reactions 0 assignees View on GitHub
Feature
Dominant language
C#
Stars
770
Forks
248
Avg merge
2d 22h
Merged PRs (30d)
51

Description

Part of #5420. Phase 2 — behavior-neutral consumer migration.

Migrate `Sentry.AspNetCore` to the resolved config:

- `src/Sentry.AspNetCore/ScopeExtensions.cs:40-52` — user factory gating → `UserInfo`.
- `:140-151` — cookie/header capture → `Cookies` / `HttpHeaders.Request` behaviors (shared filter utility).
- `:155-159` — `REMOTE_ADDR` client IP → `UserInfo`.
- `:136` — query string → `UrlQueryParams` behavior.
- `:170-186` + `RequestBodyExtractionDispatcher` — body capture → `HttpBodies` flag for *whether*; `MaxRequestBodySize` keeps governing *size* (integration option takes precedence).
- `src/Sentry.AspNetCore.Grpc/ProtobufRequestExtractionDispatcher.cs` — same gating for protobuf bodies.
- `src/Sentry.OpenTelemetry/AspNetCoreEnricher.cs:11-20` — user factory gating → `UserInfo`.

Contributor guide

Open the contributing guide

Research direction

Start with the resolved configuration and the listed ranges in src/Sentry.AspNetCore/ScopeExtensions.cs, then inspect RequestBodyExtractionDispatcher and src/Sentry.AspNetCore.Grpc/ProtobufRequestExtractionDispatcher.cs. Also review src/Sentry.OpenTelemetry/AspNetCoreEnricher.cs:11-20. Done means each migration uses the specified behavior flags, MaxRequestBodySize precedence remains intact, and behavior stays neutral.

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
Clearly specified
Newbie friendliness
62/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.