-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
TelemetryConfigration.Active deprecated #36394
Comments
@eivinskr Thank you for your feedback. I will investigate and get back to your shortly. |
@eivenskr In the doc for .NET Core there is also a note that I will now proceed to close this thread. Please tag me if you need to discuss this further. |
Ahh, I have two things I still find weird if you don't mind discussing it @SamaraSoucy-MSFT.
But as you mention, I don't get why that should be a problem if |
The Application Insights package is used in both Framework and Core. The .NET Core SDK has it as a dependency.
If you are setting up the configuration somewhere else, like through DI, using Active creates a second instance of the configuration and can cause conflicts. This is exactly the problem that is happening in Core apps. |
Thank you for clearing up the confusion! |
TelemetryConfigration.Active is deprecated with ApplicationInsights SDK 2.11-beta1 as described here, so the first example under using config file
should maybe reflect that?
Could be done similar to how TelemetryProcessorChainBuilder is described here
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.
The text was updated successfully, but these errors were encountered: