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

scripts: note to further investigate in download_blob_from_peer #3354

Merged
merged 2 commits into from
Jul 19, 2021

Conversation

belikor
Copy link
Contributor

@belikor belikor commented Jul 7, 2021

Currently lbrynet blob get <hash> does not work to download single blobs which are not already present in the system.
The function locks up and never returns.
It only works for blobs that are in the blobfiles directory already.

This bug is reported in lbryio/lbry-sdk, issue #2070.

Since this script does in fact allow downloading an arbitrary blob, we may decide to copy parts of it and add them to lbry.extras.daemon.daemon.jsonrpc_blob_get in order to solve the previous issue.

Currently `lbrynet blob get <hash>` does not work to download
single blobs which are not already present in the system.
The function locks up and never returns.
It only works for blobs that are in the `blobfiles` directory
already.

This bug is reported in lbryio/lbry-sdk, issue lbryio#2070.

Maybe this script can be investigated, and certain parts
can be added to `lbry.extras.daemon.daemon.jsonrpc_blob_get`
in order to solve the previous issue, and finally download
single blobs from the network (peers or reflector servers).
@eukreign eukreign assigned jackrobison and unassigned eukreign Jul 19, 2021
@lyoshenka lyoshenka merged commit 406070a into lbryio:master Jul 19, 2021
@lyoshenka
Copy link
Member

thanks @belikor 👍

@belikor belikor deleted the note-download-blob-peer branch July 19, 2021 22:48
@belikor belikor mentioned this pull request Jul 20, 2021
10 tasks
@eukreign eukreign added area: other type: improvement Existing (or partially existing) functionality needs to be changed labels Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: other type: improvement Existing (or partially existing) functionality needs to be changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants