Conversation
|
bruno-garcia
left a comment
There was a problem hiding this comment.
Love the idea of this entry point to start the project!!
src/Sentry.AspNetCore.Blazor.WebAssembly/WebAssemblyHostBuilderExtensions.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
@bitsandfoxes it looks like the even that comes through to Sentry is being marked as Handled even though the exception is not caught... but I think we have a separate issue to track this as it's an issue with our logging integration rather than our blazor wasm integration:
The other thing that is a bit confusing is that the Nav menu is titled "Throw, Catch and Capture" even though the code is just doing a throw.
I had to look at the code to work out that this was a nav menu actually... initially I though that clicking on that link would throw, catch and capture and exception (one with handled=true) and that clicking on the button would throw and unhandled exception... demonstrating that the integration will catch both kinds of exceptions.
So overall, the integration itself looks great... but the Sample project is quite confusing. I think it's worth tweaking the UI in the sample project to make this less confusing.



Resolves #2021
Docs: getsentry/sentry-docs#10344