Fix ITelemetryModule singleton registration to support presence of keyed services #777
Annotations
10 warnings
|
Build:
BASE/src/Microsoft.ApplicationInsights/Extensibility/Implementation/Endpoints/EndpointMetaAttribute.cs#L10
Avoid unsealed attributes (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1813)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/TelemetryClient.cs#L313
Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/TelemetryClient.cs#L284
Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/DataContracts/ExceptionTelemetry.cs#L54
Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/Extensibility/Implementation/BroadcastProcessor.cs#L35
Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/Extensibility/Implementation/BroadcastProcessor.cs#L26
Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/Extensibility/TelemetryConfiguration.cs#L125
'active == null' is always 'true'. Remove or refactor the condition(s) to avoid dead code. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1508)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/DataContracts/ExceptionTelemetry.cs#L172
Prefer a 'TryGetValue' call over a Dictionary indexer access guarded by a 'ContainsKey' check to avoid double lookup (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1854)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/Extensibility/OperationCorrelationTelemetryInitializer.cs#L23
In member void OperationCorrelationTelemetryInitializer.Initialize(ITelemetry telemetryItem), change parameter name telemetryItem to telemetry in order to match the identifier as it has been declared in void ITelemetryInitializer.Initialize(ITelemetry telemetry) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1725)
|
|
Build:
BASE/src/Microsoft.ApplicationInsights/Metrics/Implementation/AutocollectedMetricsExtraction/TraceMetricsExtractor.cs#L94
Change type of variable 'dimensionNames' from 'System.Collections.Generic.IList<string>' to 'System.Collections.Generic.List<string>' for improved performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1859)
|
Loading