Altinn / Altinn/app-lib-dotnet

Create Roslyn analyzer to obsolete use of `TelemetryClient`?

Open
#628 1 comment 0 reactions 0 assignees View on GitHub
feature
Dominant language
C#
Stars
8
Forks
27
Avg merge
1h 21m
Merged PRs (30d)
7

Description

We should obsolete `TelemetryClient` use, since we want to remove it in the next major release in favor of OpenTelemetry and `System.Diagnostics`. Since this is just a DI registered service, we could create an analyzer to emit warning diagnostics with guidance on how to use OpenTelemetry instead.

We could also use this analyzer for other things in the future

Contributor guide

Open the contributing guide

Research direction

No files, tests, or entry points are named in the issue; begin by locating the C# project and its dependency-injection registration for TelemetryClient. Define the analyzer scope and warning guidance for replacing TelemetryClient with OpenTelemetry and System.Diagnostics, with completion marked by diagnostics that cover the agreed usages and guidance for migration.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
observability-sre, tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.