-
-
Notifications
You must be signed in to change notification settings - Fork 206
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move tunnel functionality into into Sentry.AspNetCore #1645
Conversation
Questions
|
nope |
We can add the NuGet package obsolete on the gallery once we ship this. |
@mattjohnsonpint @bruno-garcia one complexity of this is for people using |
Tunneling is useful, but completely optional. We didn't have a ASP.NET (non-core) tunnel before, so it's not an issue at the moment. If it becomes higher priority, we could consider adding one to |
SentryTunnelMiddleware
intoSentry.AspNetCore
Sentry.Tunnel
toSentry.AspNetCore
Sentry.Tunnel.SentryTunnelMiddleware
by inheriting fromSentry.AspNetCore.SentryTunnelMiddleware
Sentry.Tunnel.SentryTunnelMiddleware
Sentry.Tunnel.Tests
intoSentry.AspNetCore.Tests
[Collection("SentryTunnelCollection")]
fromIntegrationsTests
SentryTunnelingApplicationBuilderExtensions
intoSentryWebHostBuilderExtensions
SentryTunnelingApplicationBuilderExtensions
toSentryWebHostBuilderExtensions
SentryTunnelingApplicationBuilderExtensions
and points toSentryWebHostBuilderExtensions
Sentry.Tunnel
on nuget.fixes #1141