tree 1fef71da9b4c970a88fb32906314c63889d3ec60
parent b0f2423e17fbd1028f918af6b26bf6cc6331d66c
author Hasnain Lakhani <m.hasnain.lakhani@gmail.com> 1754194327 -0700
committer GitHub <noreply@github.com> 1754194327 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJojuGXCRC1aQ7uu5UhlAAAfUMQAI3V4czFrZTWaEpPM1hlCDUr
 kPKolnUsj9ldaZw3ka7ssHcTdefpRWzzfFMUMRvzwwLUwM2tQjJKtwJztxMt4dV1
 t8zSY/aeiFkSOpy8lQC+F0hvvE0ncFPlun4HRgaFfVAnRUj/JzcGywohQ4zE5Ejc
 mOg0+4ML2FDsgEp/hdzhwlVersdl90j0rsWjk9DPGWEEWbRSpz0cPQllJA6+Duck
 ejYY/gQ84+1PTomiOsMBNviP7r9z+6CNyFF9FOyZUBmZQnYXTIIBPF+hzq6iDqrE
 KtCQcvop07jmrqB5awBr4juq+B+pJO6prj6oaXNfnIFfn35DWs1L3Yiaz00+6bBW
 akgJiGwbe1bK7hBhOf5/0pGT24ATjKXViNwC6z9rGyKpVWabgXphvLylPPn/mxl7
 nqZHKCxrAt3s6YALR9p1r+UfNuxiSQWxGSaJ7zfRTXTe28WLgeebzx4ntDWb3Rk1
 4YrylfeMd+r8jeQGKtzB4lLfN5l4KZVSgcIuv4TsVhwxaDWI+JqYvCoJT+0JV+In
 PPdv9yQsFsPpW7J+bLZos3HNjedo7f89sEkHzfhbO68xR27I4MlVukuCRKd7394o
 p0QMk+Fsmxb8sFOx25aGs8/UoHNBn6B5yMh1318r2BCJJJ/kFiCvPy3xF1+PTGb5
 RqiI5k08blM9lGvZLJuZ
 =SOos
 -----END PGP SIGNATURE-----
 

Fix UUID handling in Rust Thrift compact protocol (#3179)

This commit fixes two bugs in the Rust Thrift implementation that prevented
UUID fields from working correctly with the compact protocol:

1. Missing UUID case in skip_till_depth function
   - The skip function in lib/rs/src/protocol/mod.rs did not handle TType::Uuid
   - This caused "cannot skip field type Uuid" errors when trying to skip UUID fields
   - Fixed by adding: TType::Uuid => self.read_uuid().map(|_| ())

2. Incorrect UUID serialization in compact protocol
   - The compact protocol was using read_bytes()/write_bytes() for UUIDs
   - These methods include a length prefix, but UUIDs should be transmitted as
     raw 16-byte values per the Thrift specification
   - This caused "don't know what type: 15" errors during cross-language tests
   - Fixed by reading/writing UUID bytes directly without length prefix
   - Implementation now matches the binary protocol's approach for consistency

The bug manifested as failures in cross-language tests between Go and Rust
when using compact protocol. The error occurred because Go correctly expected
raw 16-byte UUID values while Rust was incorrectly adding a length prefix.

Testing:
- Verified all go-rs and rs-go cross tests now pass with compact protocol
- Tests confirm UUID values are correctly serialized and deserialized
- Both buffered and framed transports work correctly