Skip to content

TELCU-26 Closing connection when partial messages break parser - #12

Open
genseric-ghiro wants to merge 1 commit into
forkfrom
genseric/telcu-26-stream-desync
Open

TELCU-26 Closing connection when partial messages break parser#12
genseric-ghiro wants to merge 1 commit into
forkfrom
genseric/telcu-26-stream-desync

Conversation

@genseric-ghiro

Copy link
Copy Markdown

Summary

  • Currently, we hold a single connection per address. When a corrupted arrives, parser may break and doesn't recover for the rest of the connection's lifetime. We still read messages at the kernel level, but we don't process them, which makes existing requests hang and new requests unprocessable. This PR fixes this issue in the following way:
  • if there's a partial message detected for a certain amount of time (10s default), close the connection, and reopen a new one
  • if there's a partial message detected for a certain amount of bytes (65MB), close the connection, and reopen a new one
  • allow multiple connections per address (improvement)

Tests

  • Unit tests

@genseric-ghiro
genseric-ghiro requested a review from a team August 31, 2026 16:44
@genseric-ghiro genseric-ghiro self-assigned this Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant