getsentry / getsentry/sentry-dotnet
DataCollection phase 2: migrate Sentry.AspNet (System.Web)
- 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.AspNet` (System.Web) to the resolved config:
- `src/Sentry.AspNet/Internal/SystemWebRequestEventProcessor.cs:62-64` — cookie header → `Cookies` behavior.
- `:70-92` — `UserHostAddress` IP, `identity.Name`, claims → `UserInfo`.
- `:96-99` — request body → `HttpBodies` (with `MaxRequestBodySize` precedence).
- `:50` — query string → `UrlQueryParams`.
- `src/Sentry.AspNet/HttpContextExtensions.cs:126-129` — transaction request cookies → `Cookies`.
Contributor guide
Research direction
Start with #5420 to understand the resolved config, then read the listed sections of src/Sentry.AspNet/Internal/SystemWebRequestEventProcessor.cs and src/Sentry.AspNet/HttpContextExtensions.cs. Migrate the specified query, cookie, client, identity, claims, and request-body data to the named configuration while preserving behavior; verify each listed location is covered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100