Comments (8)
Tracking this through the code, it looks like once the queued payments are removed to be sent over to rpc sendmany, we lose all record of the transaction. If any of the payments fail, like on RPC_WALLET_UNLOCK_NEEDED, we throw that error, but as I far as I can see, there's no way we handle a failed payment or retry. Is the solution to keep all queued payments stored on client after they've been sent, only removing after we get a successful response? And should we be holding up any downloading of the file until after we get the successful response, or are we assuming it's eventually going to succeed?
from lbry-sdk.
Any update? I see this a major impediment for the future media providers - free downloads...
from lbry-sdk.
@ELM4Ever Forwarded to dev team, can also confirm this works
from lbry-sdk.
Does this only download the encrypted data, or does it also unlock the decryption key? If the latter, there's something wrong on the seller side, not just the client.
from lbry-sdk.
@sonatagreen the decryption key is stored in the stream descriptor blob, which the host doesn't necessarily provide. So nothing is going wrong on the seller side.
This is a bug, in that the program shouldn't let you download something without paying. However, it will be a more complex solution down the road that gets this locked down to the point where editing some code (or stumbling on a bug like this) won't let you bypass payment.
from lbry-sdk.
Is this still a bug?
from lbry-sdk.
related to #719
from lbry-sdk.
fixed
from lbry-sdk.
Related Issues (20)
- May we have an updated version of documentation on daemon settings ?
- thumbnails in LBRY
- Allow multiple blob_dir HOT 2
- claim list with full text search
- Various tests failing when run on Ubuntu 22.04.1: "ValueError: unsupported hash type ripemd160" HOT 4
- Check tx size when creating transactions (or use batching for all txes)
- invalid transaction: 0 output tx created, fails to broadcast HOT 4
- Can't load lbrynet on Linux Mint HOT 9
- easy way to find + delete orphaned reposts
- how to properly link the wallet with lbry account wallet HOT 6
- Workflow generating lots of warnings: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. HOT 1
- bittrex timeout prevents startup? HOT 1
- Add field param
- No Channel For Android HOT 1
- Some files get incorrect duration HOT 2
- Opm
- Could you dev option "private" or "secret"?
- Python 3.10: wait_for() got an unexpected keyword argument 'loop'
- Questions about sha256 and salt used
- Using wrong part of the result to check compatible version
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from lbry-sdk.