Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use unordered datachannel in WebRTC #22

Open
ccaapton opened this issue Apr 25, 2024 · 3 comments
Open

Use unordered datachannel in WebRTC #22

ccaapton opened this issue Apr 25, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request question Further information is requested

Comments

@ccaapton
Copy link

Hi, I noticed that weron is using the default datachannel configuration, which is ordered thus not suitable for lossy packet transmission.

Would you add more configuration, perhaps end user configurable, to allow unordered datachannel as the underlying transport?

@NNdroid
Copy link

NNdroid commented May 24, 2024

I think this should not be feasible. First of all, IP data packets are usually sent in fragments under tun/tap. This will cause the MTU of the tunnel to be a reasonable value, otherwise communication cannot be carried out. This will not happen when using strong parity flow. The problem is, it can be viewed as streams, not packets.

@ccaapton
Copy link
Author

Stream over stream is just another version of tcp over tcp, which is very problematic. The user should able able to choose a smaller mtu, or rely on tcp mss discovery mechanism.

And why do we need strong parity on the first place?

@pojntfx pojntfx added the question Further information is requested label Jul 30, 2024
@pojntfx
Copy link
Owner

pojntfx commented Jul 30, 2024

I'm definitely open to taking another look at this once the transition to an asynchronous RPC protocol is done (see https://github.com/pojntfx/panrpc), but with the current RPC system this would not be possible.

@pojntfx pojntfx self-assigned this Jul 30, 2024
@pojntfx pojntfx added the enhancement New feature or request label Jul 30, 2024
@pojntfx pojntfx changed the title Use unordered datachannel in webrtc Use unordered datachannel in WebRTC Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants