Eric Sampson
Eric Sampson
Oops sorry I missed seeing the request for comments
it feels to me like the only entity who knows for sure if an exception sent to `CaptureException` is actually "handled" or not is the caller. So should this be...
> It's possible for the caller to set handled to true/false by setting the "mechanism" for the exception before capturing it via `exception.SetSentryMechanism(mechanism, description, handled: false);` Is that even documented...
> If a user captures an exception via CaptureException isn't it handled implicitly? @bitsandfoxes do you mean semantically? or are you talking about the implementation
@bitsandfoxes re this statement: > If a user calls CaptureExeception that event should be marked with handled: true. IMO this is not correct. Only the code author that called `CaptureException`...
I'm very confused. > This issue descibes an issue with: An integration captures an exception, it gets processed by the SDK and the event processor (I think wrongly) assumes that...
> All of which I think takes us back to what @bitsandfoxes https://github.com/getsentry/sentry-dotnet/issues/3383#issue-2312683216 (quite well, and without all the confusion that I added - apologies for that). right, that's all...
> So we're saying user's expectations, in this case, would be for Handled to default to true? I think it's more about matching the Sentry intent than end-user expectations, to...
Either of the suggestions that bruno made sound good. Especially if it doesn't require/cause any user-facing changes (ie have the logging libraries do the right thing automatically)
Could it also support StackExchange.Redis, since that is such a super-popular package in .NET ?