[THRIFT-5299] rs implementation compact protocol seq_id should not use zigzag encoding.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.13.0
• Fix Version/s: 0.14.1
• Component/s: Rust - Library
• Assignee: Allen George
• Reporter: shuo li
• Created: 28/Oct/20 04:11
• Updated: 22/Feb/21 01:18
• Resolved: 22/Feb/21 01:18

Description

While reviewing code, I discovered a bug in compact protocol.

The seq_id in message header is decoded with integer-encoding's i32, which uses zigzag decode implicitly. 

 

 

EDIT: correct the title and desc.

Issue Links

• links to: GitHub Pull Request #2336