header values are be expected to be W3C baggage encoded

[zeitlinger]

Blocked by open-telemetry/opentelemetry-specification#3832

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4407ee7) 90.99% compared to head (5c4b4dd) 90.99%.
Report is 7 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #6164      +/-   ##
============================================
- Coverage     90.99%   90.99%   -0.01%     
- Complexity     5646     5673      +27     
============================================
  Files           619      620       +1     
  Lines         16443    16542      +99     
  Branches       1663     1683      +20     
============================================
+ Hits          14963    15052      +89     
+ Misses         1017     1013       -4     
- Partials        463      477      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zeitlinger]

@jack-berg can this error be ignored?
It's for https://github.com/open-telemetry/opentelemetry-java/pull/6164/files#diff-9a1bf844e080131caefb308fe4cf25d03e85322a57c15241b8872c7541638b45R101

[jack-berg]

@jack-berg can this error be ignored?

Yes and no. The codecov target is a goal, not a strict requirement. Sometimes its not practical to have all the code paths covered. Although in this case it seems like it should be straight forward to trigger than exception, no?

[jack-berg]

Thanks for the bug fix. Looks good, but let's try to cover the exception case.

[zeitlinger]

Thanks for the bug fix. Looks good, but let's try to cover the exception case.

Fond a way 😄

[jack-berg]

Waiting to merge until its clear whether this is going to be considered a breaking change or a bug fix based on this spec conversation: open-telemetry/opentelemetry-specification#3832