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

Support resuming downloads #64

Closed
transitive-bullshit opened this issue May 29, 2014 · 3 comments
Closed

Support resuming downloads #64

transitive-bullshit opened this issue May 29, 2014 · 3 comments

Comments

@transitive-bullshit
Copy link
Member

@transitive-bullshit transitive-bullshit commented May 29, 2014

Now that webtorrent is fs-backed, when a torrent is added, we should check for existing files in the destination directory, verifying piece hashes to properly initialize the storage state, and only then proceed with the download. If the torrent already exists and verifies as completed, no downloading should take place, just seeding.

webtorrent.app will take care of persisting torrent metadata across runs, along with known peers.

@transitive-bullshit

This comment has been minimized.

Copy link
Member Author

@transitive-bullshit transitive-bullshit commented May 29, 2014

Note that I'm opening this to track the issue but expect to have it implemented sometime this week.

@transitive-bullshit

This comment has been minimized.

Copy link
Member Author

@transitive-bullshit transitive-bullshit commented May 31, 2014

This is now implemented pending approval across my PRs to bittorrent-client https://github.com/feross/bittorrent-client/pull/20 and webtorrent #63.

@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Jun 12, 2014

Closing because this has been implemented for a while.

@feross feross closed this Jun 12, 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.