question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Revert adding Sentry to global usings

See original GitHub issue

The latest Release 3.14.0 comes with the following feature:

Add Sentry to global usings when ImplicitUsings is enabled (<ImplicitUsings>true</ImplicitUsings>) (https://github.com/getsentry/sentry-dotnet/pull/1398)

We do have implicit usings enabled, with the intention was that only the often used “standard”-namespaces are added there. Sentry on the other hand is a namespace which we use exactly once in our ASP.NET Core API projects: when configuring Sentry on startup.

For me as an “end user” of this library it does make absolutely no sense to have Sentry in the global usings.

Main issue:
The main conflict here is Sentry.User. I guess a User is not a really uncommon name for a class. Now, with the latest update we have a lot of ambiguous references between Sentry.User and our Domain-User. As a workaround I’d have to explicitly reference our Domain-User in a ton of classes over multiple projects. Which I really don’t want to do.

So I vote for removing Sentry from global usings again.

Issue Analytics

  • State:closed
  • Created 2 years ago
  • Reactions:1
  • Comments:5 (4 by maintainers)

github_iconTop GitHub Comments

1reaction
SimonCroppcommented, Feb 17, 2022

wouldnt this suffice as a opt-out ?

 <ItemGroup>
    <Using Remove="Sentry" />
0reactions
bruno-garciacommented, Feb 17, 2022

We discussed and agreed to revert this feature, until we release 4.0 that will include renaming some types prone to conflict. Lets spin off a new issue to track the class renames and adding this feature back in.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Troubleshooting for ASP.NET Core
The message currently appears as information only, though we may change it to a ... then Sentry will be added to the current...
Read more >
Sentry SDK with Asp.Net 4.6.1 Not working
I have an Asp.Net app, using .net 4.6.1 version. When I installed Sentry nuget and used Sentry SDK, nothing working, no events are...
Read more >
getsentry/sentry-cli: A command line utility to work with ...
This is a Sentry command line client for some generic tasks. Right now this is primarily used to upload debug symbols to Sentry...
Read more >
Guidelines for Performance Monitoring
This document covers how SDKs should add support for Performance Monitoring with Distributed Tracing . This should give an overview of the APIs...
Read more >
Error Reporting Using Sentry in Flutter
However, In this article you will learn how to monitor your applications and potential errors or bugs and report them using Sentry.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found