opentelemetry-dotnet icon indicating copy to clipboard operation
opentelemetry-dotnet copied to clipboard

#3490 follow up (Fix asp.net core middleware activity modification issue)

Open vishweshbankwar opened this issue 3 years ago • 1 comments

Follow up to #3490

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

  • [x] Appropriate CHANGELOG.md updated for non-trivial changes
  • [ ] Design discussion issue #
  • [ ] Changes in public API reviewed

vishweshbankwar avatar Jul 27 '22 22:07 vishweshbankwar

Codecov Report

Merging #3498 (5038cc9) into main (99b49b6) will decrease coverage by 0.03%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3498      +/-   ##
==========================================
- Coverage   87.46%   87.42%   -0.04%     
==========================================
  Files         278      278              
  Lines       10082    10087       +5     
==========================================
+ Hits         8818     8819       +1     
- Misses       1264     1268       +4     
Impacted Files Coverage Δ
...tation.AspNetCore/Implementation/HttpInListener.cs 90.18% <100.00%> (+0.31%) :arrow_up:
...Listener/Internal/PrometheusExporterEventSource.cs 16.66% <0.00%> (-11.12%) :arrow_down:
...ZPages/Implementation/ZPagesExporterEventSource.cs 56.25% <0.00%> (-6.25%) :arrow_down:
....Prometheus.HttpListener/PrometheusHttpListener.cs 78.66% <0.00%> (-4.00%) :arrow_down:
...ter.ZPages/Implementation/ZPagesActivityTracker.cs 97.14% <0.00%> (-2.86%) :arrow_down:
src/OpenTelemetry/Logs/OpenTelemetryLogger.cs 88.88% <0.00%> (+2.22%) :arrow_up:
...tation/OpenTelemetryProtocolExporterEventSource.cs 85.00% <0.00%> (+10.00%) :arrow_up:

codecov[bot] avatar Jul 28 '22 00:07 codecov[bot]