-
-
Notifications
You must be signed in to change notification settings - Fork 230
ScopeExtensions.Populate should not be public #4352
Copy link
Copy link
Closed
Labels
.NETPull requests that update .net codePull requests that update .net codeRepository MaintenanceTaskpublic APIAdditions/modifications to, or removals from, the public API surface area.Additions/modifications to, or removals from, the public API surface area.
Milestone
Metadata
Metadata
Assignees
Labels
.NETPull requests that update .net codePull requests that update .net codeRepository MaintenanceTaskpublic APIAdditions/modifications to, or removals from, the public API surface area.Additions/modifications to, or removals from, the public API surface area.
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
Done
ScopeExtensions.Populateis currently public.sentry-dotnet/src/Sentry.AspNetCore/ScopeExtensions.cs
Line 25 in 4a764c9
We call this from the SentryMiddleware:
sentry-dotnet/src/Sentry.AspNetCore/SentryMiddleware.cs
Line 258 in 6e50eaa
However SDK users shouldn't need to call this so it should be internal.
Breaking changes so let's do it in the next major release.