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

Think about settlement beneficiaries other than the channel participants #24

Open
loredanacirstea opened this issue Mar 9, 2018 · 2 comments
Labels
core:smart contracts Core Smart Contracts

Comments

@loredanacirstea
Copy link
Contributor

Do we need functionality to have a beneficiary of settlement payouts?
Example: embedded devices with their own privatekey that are funded by human user with a different privatekey.
This can also apply to third party services that can provide token deposits on behalf of a channel participant for easier onboarding.

@loredanacirstea loredanacirstea added the core:smart contracts Core Smart Contracts label Mar 9, 2018
@LefterisJP
Copy link
Contributor

I would say not at the moment. If the embedded device is owned by the human user they can always just forward the funds.

I mean it would be convenient perhaps but I would rather keep the protocol as simple as possible and keep only to the "strictly required" at the moment.

@hackaugusto
Copy link
Contributor

I see three options:

  • Define the beneficiary address in at the channel creation (bad, currently we required only one node to open a channel, this would required a protocol for opening channels. Additionaly, chaging the beneficiary would require a on-chain transaction)
  • Additional field as part of the balance proof (doesnt change channelOpen, but increases the gas cost for storing the beneficiary field on uncooperative close. May be good for upgrading the beneficiary address, with the caveat that older balance proofs are still valid, and under bad behavior an old address may receive the funds)
  • As part of the balance proof's additional_hash. (almost the same as the previous option, but we trade storage for code complexity and additional hash computations)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core:smart contracts Core Smart Contracts
Projects
None yet
Development

No branches or pull requests

4 participants