newrelic / newrelic-java-agent

The New Relic Java agent
Apache License 2.0
202 stars 143 forks source link

Prevent duplicate DT headers for grpc #1783

Closed kanderson250 closed 7 months ago

kanderson250 commented 8 months ago

Resolves #1768

This PR addresses a bug with grpc apps using distributed tracing. Previously our grpc instrumentation allowed duplicate DT headers, which caused DT payloads to be rejected by downstream services. This is now prohibited.

Changes:

codecov-commenter commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.83%. Comparing base (a84c048) to head (c5ff850). Report is 863 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1783 +/- ## ========================================= Coverage 70.82% 70.83% - Complexity 9934 9946 +12 ========================================= Files 825 826 +1 Lines 39826 39875 +49 Branches 6037 6043 +6 ========================================= + Hits 28207 28245 +38 - Misses 8891 8903 +12 + Partials 2728 2727 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.