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

What sort of congestion control is in use? #142

Closed
dtaht opened this issue Sep 27, 2014 · 2 comments
Closed

What sort of congestion control is in use? #142

dtaht opened this issue Sep 27, 2014 · 2 comments

Comments

@dtaht
Copy link

@dtaht dtaht commented Sep 27, 2014

Torrent normally uses a modified reno algorithm with reduced slow start and a delay based approach to backing off (ledbat). From what I read here, webtorrent is currently using straight TCP to do it's work, no modifications, and doesn't include a uTP implementation?

@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Oct 7, 2014

In node, WebTorrent uses tcp sockets for peer connections, though as I understand it, uTP would offer better performance while making WebTorrent a better network citizen (by backing off to TCP traffic). There's an open issue to implement uTP: #68 Note, the cardinal rule of open source applies here: "If you see a job that needs doing, it's your job. 👍"

In the browser, WebTorrent uses webrtc data channels in reliable mode (ordered, guaranteed delivery) which emulates the semantics of tcp sockets, for parity with the node implementation. I'm open to investigating alternatives, and willing to accept modifications that improve perf.

@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Oct 7, 2014

Closing this issue since an issue is already open for uTP support (#68). But you can feel free to continue discussing.

@feross feross closed this Oct 7, 2014
@lock lock bot locked as resolved and limited conversation to collaborators May 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.