Comments (4)
I'm sorry that wasted your time.
I hope it's clear why we filter out Sentry namespace though since this avoids an internal error in the SDK getting into an infinite loop.
Besides documenting this behavior, I'd also suggest we log a debug message.
This way, for those who didn't spot the note in the docs, by enabling Debug mode it will be clear what's going on.
Thanks for raising this.
from sentry-dotnet.
Just wondering whether there's potentially another way forward.. Thinking out loud here.
Is it true to say that any Sentry namespaces that could cause infinite loops are either within the Extensions.Logging project itself or a referenced assembly? If so, I'm just wondering whether you could say "it begins with Sentry and it's in this assembly or one of this assembly's references".
from sentry-dotnet.
An error could come from Sentry
package and end up in a global handler defined by the application which calls LogError
. Or from a package that wraps MEL like Sentry.AspNetCore
.
We could have this filter prefix configurable, with Sentry there by default.
At least this is only an issue when creating test apps named 'SentryTest' or something.
Perhaps a simple fix would be to simply changing to:
namespace.StartsWith("Sentry.") || namespace == "Sentry"
This could be a simpler change.
from sentry-dotnet.
@cpwood I was explicit about this behaviour in the docs now:
from sentry-dotnet.
Related Issues (20)
- User ID is left blank on Android HOT 2
- High memory usage from ISpan[] and SpanTracer HOT 4
- Sentry assembly produces trim warnings when compiling AOT HOT 1
- System.IO.IOException in DebugStackTrace
- Hangfire not in cron HOT 1
- Add logs to `metrics`
- Remove FirstChanceException workaround
- Try to populate DeviceInfo if/when sentry-native is available
- Add the Refit Exception Content to Sentry HOT 4
- Improve OutOfMemory insight
- Incorrect stack traces for UWP in .NET Native HOT 5
- Filter out health check transactions HOT 6
- Cocoa Issues
- Enhanced Stack Frames with Native AOT HOT 4
- Crash in release mode when using the Logging extensions only HOT 14
- `scope.User = null` shows warning of CS8625: Cannot convert null literal to non-nullable reference type. HOT 6
- Add a benchmark action
- Make span eligible for `Requests` HOT 2
- install instructions for profiling have the wrong version HOT 4
- Sentry dotnet SDK is blowing away User.IpAddress if we set it explicitly and SendDefaultPii is true HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sentry-dotnet.