http-instrumentation: fix propagation errors when using Expect header #481

[vmarchaud]

No description provided.

[codecov-io]

Codecov Report

Merging #482 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #482      +/-   ##
==========================================
- Coverage   94.83%   94.69%   -0.15%     
==========================================
  Files         150      150              
  Lines       10153     9745     -408     
  Branches      752      737      -15     
==========================================
- Hits         9629     9228     -401     
+ Misses        524      517       -7

Impacted Files	Coverage Δ
src/trace/propagation/noop-propagation.ts	37.5% <0%> (-12.5%)	⬇️
src/internal/util.ts	90% <0%> (-10%)	⬇️
src/common/validations.ts	88.23% <0%> (-7.22%)	⬇️
src/tags/validation.ts	92.85% <0%> (-7.15%)	⬇️
src/trace/model/no-record/no-record-span.ts	54.54% <0%> (-1.02%)	⬇️
src/tags/propagation/text-format.ts	94.11% <0%> (-0.89%)	⬇️
src/tags/propagation/variant-encoding.ts	95% <0%> (-0.84%)	⬇️
src/trace/model/tracer.ts	85.71% <0%> (-0.26%)	⬇️
src/index.ts	100% <0%> (ø)	⬆️
src/http-stats.ts	100% <0%> (ø)	⬆️
... and 45 more

---

Continue to review full report at Codecov.

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

[draffensperger]

LGTM with minor nits

[vmarchaud]

@draffensperger I've rewritten the explanation and removed the link to the google trace agent's PR

[draffensperger]

Looks good, thanks!

[vmarchaud]

Done @mayurkale22

[mayurkale22]

Thanks for the contribution @vmarchaud