wboayue / rust-ibapi / 22126629475

86%

Build Type

push

github

Committed by web-flow

Commit Message

fix: skip unexpected messages on shared channels without retry limit (#418)

* fix(async): release order-update stream ownership on subscription drop

* fix: skip unexpected messages on shared channels without retry limit

UnexpectedResponse errors from shared broadcast channels are
wrong-channel messages (e.g., OrderStatus from order updates arriving
on the OpenOrders channel), not actual decode failures. Previously
these counted toward the 10-retry limit, causing subscriptions to
fail when enough stray messages accumulated in the ring buffer.

Change UnexpectedResponse from ProcessingResult::Retry (counted) to
ProcessingResult::Skip (silent discard). Stray messages are now
skipped at trace log level without incrementing the retry counter,
so subscriptions never fail due to cross-channel message noise.

Add ProcessingResult::Skip variant and test proving >10 unexpected
messages are tolerated without subscription failure.

---------

Co-authored-by: Wil Boayue <wil@wsbsolutions.com>

Run Details

3 of 8 new or added lines in 3 files covered. (37.5%)

5 existing lines in 2 files now uncovered.

8946 of 10461 relevant lines covered (85.52%)

121.11 hits per line