Support Tag Keys for DD_TRACE_HEADER_TAGS

[link04]

Description

Adding Support for allowing customers to pass : separated values in the list of header tags which they would like to use the key instead of the current header name only.

Reviewer checklist

• Test coverage seems ok.
• Appropriate labels assigned.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.24%. Comparing base (9d93b40) to head (2f104b2).
Report is 20 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #2922   +/-   ##
=========================================
  Coverage     78.24%   78.24%           
  Complexity     2526     2526           
=========================================
  Files           135      135           
  Lines         14400    14400           
  Branches        989      989           
=========================================
  Hits          11267    11267           
  Misses         2591     2591           
  Partials        542      542           

Flag	Coverage Δ
appsec-extension	68.36% <ø> (ø)
tracer-php	82.09% <ø> (ø)

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

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d93b40...2f104b2. Read the comment docs.

[pr-commenter]

Benchmarks [ tracer ]

Benchmark execution time: 2024-11-06 19:22:18

Comparing candidate commit 2f104b2 in PR branch maximo/consistent-header-tags with baseline commit 9d93b40 in branch master.

Found 1 performance improvements and 0 performance regressions! Performance is the same for 177 metrics, 0 unstable metrics.

scenario:TraceSerializationBench/benchSerializeTrace

• 🟩 execution_time [-11.004µs; -7.496µs] or [-5.675%; -3.866%]

[bwoebi]

Implementation looks good so far, but peer service tests seems to want that you ignore these cases without the colon. (and tea tests do need some updates too).

[bwoebi]

Just styling notes, looks good now :-)

[bwoebi]

LGTM, thanks Maximo :-)