Skip to content
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

Use Azure App Insights for telemetry #490

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Conversation

christianhelle
Copy link
Owner

This pull request adds telemetry logging to both Exceptionless and App Insights.

It includes a new package reference to Microsoft.ApplicationInsights.WindowsServer and modifies the Analytics class to configure and log telemetry using the TelemetryClient. The LogFeatureUsage method has been updated to log feature usage to both Exceptionless and App Insights.

Additionally, a new SupportKeyInitializer class has been added to initialize the support key property in telemetry.

@christianhelle christianhelle linked an issue Sep 23, 2024 that may be closed by this pull request
Copy link

sonarcloud bot commented Sep 23, 2024

@christianhelle christianhelle self-assigned this Sep 23, 2024
@christianhelle christianhelle added enhancement New feature, bug fix, or request .NET Pull requests that contain changes to .NET code labels Sep 23, 2024
Copy link

codecov bot commented Sep 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.71%. Comparing base (3ca1a28) to head (e55c87c).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #490      +/-   ##
==========================================
- Coverage   94.58%   91.71%   -2.88%     
==========================================
  Files          51       51              
  Lines        2493     2607     +114     
==========================================
+ Hits         2358     2391      +33     
- Misses        108      189      +81     
  Partials       27       27              
Flag Coverage Δ
unittests 91.71% <ø> (-2.88%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@christianhelle christianhelle merged commit 65b9dfe into main Sep 23, 2024
7 of 8 checks passed
@christianhelle christianhelle deleted the azure-app-insights branch September 23, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, bug fix, or request .NET Pull requests that contain changes to .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exceptionless monthly limit exceeded in only a few days
1 participant