feat: User.Id can now be overriden (set to null) in Global mode by jamescrosswell · Pull Request #5039 · getsentry/sentry-dotnet

jamescrosswell · 2026-03-18T23:02:48Z

Resolves #4172

(MAUI) user.id is always set and can't remove it with SendDefaultPii or null #4172

github-actions · 2026-03-18T23:03:10Z

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

Breaking Changes 🛠

The _Metrics_ APIs are now stable: removed Experimental from SentrySdk, SentryOptions and IHub by Flash0ver in #5023

Features ✨

Report a new _Diagnostic_ (SENTRY1001) when a Metrics-API is invoked with an unsupported numeric type by Flash0ver in #4840

feat: User.Id can now be overriden (set to null) in Global mode by jamescrosswell in #5039

Fixes 🐛

fix: include Data set via ITransactionTracer in SentryTransaction by Flash0ver in #4148
fix: CaptureFeedback now applies event processors by jamescrosswell in #4942

Dependencies ⬆️

Deps

chore(deps): update Native SDK to v0.13.3 by github-actions in #5045
chore(deps): update Cocoa SDK to v9.7.0 by github-actions in #5015
chore(deps): update Java SDK to v8.35.0 by github-actions in #5017
chore(deps): replaced the heavy protobuf-javalite 3.25.8 dependency with a light-weight epitaph 0.1.0 alternative on Android (getsentry/sentry-java#5157) by github-actions in #5017
chore(deps): update CLI to v3.3.3 by github-actions in #5002
chore(deps): update Cocoa SDK to v9.6.0 by github-actions in #4958

Other

ref: Use .NET 6.0 ArgumentNullException throw helpers by copilot-swe-agent in #4985

_{🤖 This preview updates automatically when you update the PR.}

codecov · 2026-03-19T02:36:33Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.07%. Comparing base (c7d8f23) to head (af186b3).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5039      +/-   ##
==========================================
+ Coverage   73.87%   74.07%   +0.20%     
==========================================
  Files         506      507       +1     
  Lines       18247    18255       +8     
  Branches     3564     3568       +4     
==========================================
+ Hits        13480    13523      +43     
+ Misses       3891     3865      -26     
+ Partials      876      867       -9

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot

Pull request overview

This PR adjusts how User.Id is populated from InstallationId when Global Mode is enabled, so that applications can explicitly clear/override the value (e.g., set it to null) without it being re-applied during event enrichment.

Changes:

Add a new GlobalRootScopeIntegration default integration that sets scope.User.Id from options.InstallationId at startup (Global Mode only).
Update Enricher to not apply the InstallationId fallback in Global Mode (so user overrides aren’t re-written).
Update/extend tests and Verify snapshots to account for the new default integration and changed behavior.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
`src/Sentry/Integrations/GlobalRootScopeIntegration.cs`	New integration to set `User.Id` on the root scope in Global Mode.
`src/Sentry/Internal/Enricher.cs`	Stops applying `InstallationId` fallback when Global Mode is enabled.
`src/Sentry/SentryOptions.cs`	Registers the new integration as a default integration.
`test/Sentry.Tests/Integrations/GlobalRootScopeIntegrationTests.cs`	Adds tests covering the new integration and updated enricher behavior.
`test/Sentry.Tests/SentryClientTests.cs`	Adjusts tests around fallback `User.Id` behavior.
`test/Sentry.Tests/SentryOptionsTests.verify.cs` + `*.verified.txt`	Narrows snapshot scope to integration-registration logs and updates expected output.
`src/Sentry/PlatformAbstractions/FrameworkInfo.cs`	Adds an additional .NET Framework release key mapping for 4.8.1.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

lucas-zimerman · 2026-04-23T18:09:04Z

Thank you for the PR!

…-user-id-4172

Flash0ver · 2026-05-05T20:35:46Z

            {533320, "4.8.1"},
-            {533325, "4.8.1"}
+            {533325, "4.8.1"},
+            {533509, "4.8.1"}


question: how did you notice this?

CI just started to fail when a new servicing update is applied to the runners?

I think I ran the unit tests on my Windows box which is running the latest version of Windows and service packs, which broke the tests. Presumably those are now hitting the CI runners.

- Add Scope + SubstituteConfigureScope to the disabled test, with assertion that scope.User.Id remains null - Add hub.Received(1).ConfigureScope assertion to both enabled tests Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-authored-by: Stefan Pölz <38893694+Flash0ver@users.noreply.github.com>

- Use nameof(GlobalRootScopeIntegration) in Skip.If messages for easier navigation - Move comment to after // Arrange in CaptureEvent_UserIsNull_SetsFallbackUserId to match CaptureTransaction_UserIsNull_SetsFallbackUserId Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

> ℹ️ **Note** > > This PR body was truncated due to platform limits. This PR contains the following updates: | Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) | |---|---|---|---| | [Sentry.Maui](https://sentry.io/) ([source](https://github.com/getsentry/sentry-dotnet)) | `6.5.0` → `6.6.0` | ![age](https://developer.mend.io/api/mc/badges/age/nuget/Sentry.Maui/6.6.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Sentry.Maui/6.5.0/6.6.0?slim=true) | --- ### Release Notes <details> <summary>getsentry/sentry-dotnet (Sentry.Maui)</summary> ### [`v6.6.0`](https://github.com/getsentry/sentry-dotnet/blob/HEAD/CHANGELOG.md#660) [Compare Source](getsentry/sentry-dotnet@6.5.0...6.6.0) ##### Features ✨ - feat: propagate trace to `sentry-android` and `sentry-cocoa` by [@bitsandfoxes](https://github.com/bitsandfoxes) in [#5244](getsentry/sentry-dotnet#5244) - feat: User.Id can now be overriden (set to null) in Global mode by [@jamescrosswell](https://github.com/jamescrosswell) in [#5039](getsentry/sentry-dotnet#5039) - feat: Implement strict trace continuation by [@giortzisg](https://github.com/giortzisg) in [#4981](getsentry/sentry-dotnet#4981) ##### Fixes 🐛 - fix: return early from AddSentryOtlpExporter when DSN is the disable-SDK sentinel by [@jamescrosswell](https://github.com/jamescrosswell) in [#5247](getsentry/sentry-dotnet#5247) - fix: sync default tags to native layer by [@bitsandfoxes](https://github.com/bitsandfoxes) in [#5214](getsentry/sentry-dotnet#5214) ##### Dependencies ⬆️ ##### Deps - chore(deps): update Cocoa SDK to v9.14.0 by [@github-actions](https://github.com/github-actions)\[bot] in [#5252](getsentry/sentry-dotnet#5252) - chore(deps): update Java SDK to v8.42.0 by [@github-actions](https://github.com/github-actions)\[bot] in [#5208](getsentry/sentry-dotnet#5208) ##### Other - release: 6.6.0 by [@jamescrosswell](https://github.com/jamescrosswell) in [0140be0a](getsentry/sentry-dotnet@0140be0) - chore: update scripts/update-cli.ps1 to 3.4.3 by [@github-actions](https://github.com/github-actions)\[bot] in [#5251](getsentry/sentry-dotnet#5251) - ci: update the Update Dependencies workflow by [@Flash0ver](https://github.com/Flash0ver) in [#5175](getsentry/sentry-dotnet#5175) - chore: update modules/sentry-native to 0.14.2 by [@github-actions](https://github.com/github-actions)\[bot] in [#5229](getsentry/sentry-dotnet#5229) - chore: update...

feat: User.Id can now be overriden (set to null) in Global mode

28e9845

Resolves #4172 - #4172

jamescrosswell commented Mar 18, 2026

View reviewed changes

Comment thread src/Sentry/Internal/Enricher.cs

jamescrosswell requested a review from lucas-zimerman March 18, 2026 23:05

jamescrosswell and others added 4 commits March 19, 2026 12:14

additional core verify tests

1b15d0f

Windows verify files

7b2e38d

Fix verify files

7a7d0f6

Windows verify files

9907a6c

jamescrosswell commented Mar 19, 2026

View reviewed changes

Comment thread ...ptionsTests.Integrations_default_ones_are_properly_registered.DotNet10_0.DotNet.verified.txt

jamescrosswell commented Mar 19, 2026

View reviewed changes

Comment thread src/Sentry/Integrations/GlobalRootScopeIntegration.cs

Skip tests on mobile

2663ae0

jamescrosswell mentioned this pull request Mar 22, 2026

feat: add Attributes APIs #4882

Open

Fixed build on NetFX/Standard

abca943

jamescrosswell marked this pull request as ready for review March 23, 2026 06:42

jamescrosswell requested a review from Flash0ver as a code owner March 23, 2026 06:42

jamescrosswell requested a review from Copilot March 23, 2026 06:43

Copilot started reviewing on behalf of jamescrosswell March 23, 2026 06:43 View session

Copilot AI reviewed Mar 23, 2026

View reviewed changes

jamescrosswell and others added 2 commits March 23, 2026 20:16

Update src/Sentry/Internal/Enricher.cs

8c2d66b

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Merge branch 'main' into remove-user-id-4172

04435e6

jamescrosswell added 2 commits April 30, 2026 17:10

Improve tests

219f31e

Merge remote-tracking branch 'origin/remove-user-id-4172' into remove…

8ae30ad

…-user-id-4172

Flash0ver reviewed May 5, 2026

View reviewed changes

jamescrosswell and others added 4 commits May 6, 2026 17:28

Merge remote-tracking branch 'origin/main' into remove-user-id-4172

4d523b3

Apply suggestions from code review

3141233

Co-authored-by: Stefan Pölz <38893694+Flash0ver@users.noreply.github.com>

jamescrosswell requested a review from Flash0ver May 6, 2026 05:45

Uh oh!

Conversation

jamescrosswell commented Mar 18, 2026

Uh oh!

github-actions Bot commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Semver Impact of This PR

Breaking Changes 🛠

Features ✨

Fixes 🐛

Dependencies ⬆️

Deps

Other

Uh oh!

Uh oh!

Uh oh!

Uh oh!

codecov Bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lucas-zimerman commented Apr 23, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Flash0ver May 5, 2026

Choose a reason for hiding this comment

Uh oh!

jamescrosswell May 5, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

github-actions Bot commented Mar 18, 2026 •

edited

Loading

codecov Bot commented Mar 19, 2026 •

edited

Loading