Skip to content

Length prefixed streams#1232

Draft
giurgiur99 wants to merge 20 commits intomainfrom
debug-longer-timeout
Draft

Length prefixed streams#1232
giurgiur99 wants to merge 20 commits intomainfrom
debug-longer-timeout

Conversation

@giurgiur99
Copy link
Contributor

@giurgiur99 giurgiur99 commented Feb 26, 2026 •
edited
Loading

BREAKING CHANGE!

Changes proposed in this PR:

  • Length-prefixed (lp) sends each message as length + payload. The receiver reads the length, then exactly that many bytes, so it always knows where one message ends and the next begins.

@giurgiur99 giurgiur99 changed the title timeout Length prefixed streams Feb 28, 2026
@giurgiur99 giurgiur99 mentioned this pull request Feb 28, 2026
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexcos20 alexcos20 alexcos20 approved these changes

@bogdanfazakas bogdanfazakas Awaiting requested review from bogdanfazakas bogdanfazakas will be requested when the pull request is marked ready for review bogdanfazakas is a code owner

@denisiuriet denisiuriet Awaiting requested review from denisiuriet denisiuriet will be requested when the pull request is marked ready for review denisiuriet is a code owner

@ndrpp ndrpp Awaiting requested review from ndrpp ndrpp will be requested when the pull request is marked ready for review ndrpp is a code owner

@andreip136 andreip136 Awaiting requested review from andreip136 andreip136 will be requested when the pull request is marked ready for review andreip136 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@giurgiur99 @alexcos20