Skip to content

add support for raw tcp stream with no predefined framing#50

Open
0w3n-d wants to merge 2 commits intomainfrom
od/raw_streams
Open

add support for raw tcp stream with no predefined framing#50
0w3n-d wants to merge 2 commits intomainfrom
od/raw_streams

Conversation

@0w3n-d
Copy link

@0w3n-d 0w3n-d commented Mar 19, 2026

@0w3n-d 0w3n-d requested review from a team and vladimir-ea March 19, 2026 13:23
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@gd-0
Copy link
Contributor

gd-0 commented Mar 19, 2026

Can you add some info about the motivation here? what is this trying to solve

@0w3n-d
Copy link
Author

0w3n-d commented Mar 19, 2026

Can you add some info about the motivation here? what is this trying to solve

The current stream is very limited. It can only be used for custom protocols that follow the framing we've baked in.
The raw stream allows this to be used as the basis for any tcp backed application layer protocol. e.g. http, websockets, gRPC, and so on.

I want to use it for the handful of requests we make in the housekeeper so that I can drop the tokio part and make it a proper tile, but we can also use it in other similar contexts. It just provides a lot more flexibility for anyone that want's to use flux.

@gd-0
Copy link
Contributor

gd-0 commented Mar 19, 2026

cool makes sense. @louisponet could you please take a look?

@louisponet
Copy link
Contributor

Did you vibe some e2e test for this?

@0w3n-d
Copy link
Author

0w3n-d commented Mar 19, 2026

Did you vibe some e2e test for this?

No. I'll fix this up and add tests.

/// Read available bytes into rx_buf, returning everything received.
///
/// Loops until WouldBlock. If the buffer fills, returns what was read so
/// far — mio will re-fire a readable event while kernel data remains.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesnt' this conficlit with the comment belwo

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.

3 participants