fix(volo-thrift): unexpected UnexpectedEof Err returned by DefaultDec…

[shenyj3]

…oder::decode caused by non-standard behavior of shmipc sdk if a connection is closed normally

Motivation

Fix an unexpected UnexpectedEof log in volo-thrift/src/transport/pingpong/server.rs in ShmIPC mode if a ShmIPC connection is closed nomarlly.

The error return chain looks like:

shmipc::Stream::read()
    → read_more() // Return Error::EndOfStream
        → shmipc::compact::StreamExt::poll_read() // e.into() converts to io::Error(UnexpectedEof)
            → volo::BufReader::poll_fill_buf()
                → fill_buf().await? throw UnexpectedEof

Solution

If we can not read even a single byte at the very beginning of DefaultDecoder::decode in ShmIPC mode, which means we will catch a UnexpectedEof Err, it can be regarded as a normal connection close and just return Ok(None) to let volo to do some clean.

[codecov]

Codecov Report

❌ Patch coverage is 72.22222% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.95%. Comparing base (6e8436c) to head (b5e0aee).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
volo-thrift/src/codec/default/mod.rs	72.22%	20 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #647      +/-   ##
==========================================
+ Coverage   43.68%   43.95%   +0.27%     
==========================================
  Files         161      161              
  Lines       19679    19750      +71     
==========================================
+ Hits         8596     8681      +85     
+ Misses      11083    11069      -14     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.