[otlp] Fix incorrect length handling of Status.Description exceeding 127 bytes

[rajkumar-rangaraj]

Fixes #6117
Design discussion issue #

Changes

Please provide a brief description of the changes here.

Fixed an issue with incorrect length encoding of Status.Description when its byte length exceeds 127. The previous behavior resulted in incorrect reporting, causing serialization issues and potential payload corruption. The fix ensures proper length encoding and safe handling of longer descriptions.

Merge requirement checklist

• CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
• Unit tests added/updated
• Appropriate CHANGELOG.md files updated for non-trivial changes

[CodeBlanch]

CHANGELOG suggestion but LGTM

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.39%. Comparing base (cccf1ec) to head (401a544).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6119      +/-   ##
==========================================
- Coverage   86.39%   86.39%   -0.01%     
==========================================
  Files         257      257              
  Lines       11640    11639       -1     
==========================================
- Hits        10056    10055       -1     
  Misses       1584     1584              

Files with missing lines	Coverage Δ
...entation/Serializer/ProtobufOtlpTraceSerializer.cs	95.00% <ø> (-0.02%)	⬇️

[davidfowl]

@CodeBlanch is this in any release?

[CodeBlanch]

@davidfowl Not yet.

/cc @rajkumar-rangaraj

[rajkumar-rangaraj]

@davidfowl Plan to release the hotfix by the end of this week.