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

udp / kcp support #7

Open
untoreh opened this issue Oct 30, 2017 · 2 comments
Open

udp / kcp support #7

untoreh opened this issue Oct 30, 2017 · 2 comments
Labels

Comments

@untoreh
Copy link
Owner

untoreh commented Oct 30, 2017

the tunnel currently can forward udp.
is it worth using kcp-go lib ? probably not because it would just add another listener to tunnel traffic internally to the udp listener itself, so for using kcp an external kcp tunnel can be forwarded to the tcsplitter udp listener.
It makes more sense to implement error correction directly with the vivint/infactious lib.

@untoreh
Copy link
Owner Author

untoreh commented Oct 30, 2017

udp tunneling should be made smarter anyway because currently:

  • every packet is sent over his own tcp connection, they should be aggregated to saturate payload size
  • a udp packet cannot be larger than the payload size, should bigger packets be handled? Or should this be a notice for usage?

@untoreh
Copy link
Owner Author

untoreh commented Nov 20, 2017

current udp implementation buffers packets but data is dumped without re packeting, most udp apps rely on each packet having the right data so each shot that buffers multiple packets should carry the info to re packeting each carried payload

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant