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

LSPS1: Use a separate fee for onchain vs ln payments #91

Open
SeverinAlexB opened this issue Feb 28, 2024 · 2 comments
Open

LSPS1: Use a separate fee for onchain vs ln payments #91

SeverinAlexB opened this issue Feb 28, 2024 · 2 comments

Comments

@SeverinAlexB
Copy link
Collaborator

LSPS1 uses one field fee_total_sat to indicate how much the user needs to pay for the requested channel.

Issue: With rising onchain fees, onchain payments are exceedingly more expensive for the lsp than lightning payments. Onchain payments create its own UTXO and are therefore more expensive for the LSP to spend.

Solution: Add a different price for LN payments compared to onchain payments. This can be done directly now or in a backwards compatible way at a later point.

@SeverinAlexB SeverinAlexB changed the title LSPS1: Use a seperate fee for onchain payments vs ln payments LSPS1: Use a separate fee for onchain payments vs ln payments Feb 28, 2024
@SeverinAlexB SeverinAlexB changed the title LSPS1: Use a separate fee for onchain payments vs ln payments LSPS1: Use a separate fee for onchain vs ln payments Feb 28, 2024
@tnull
Copy link
Contributor

tnull commented Feb 28, 2024

Makes sense to me, probably best to do it now.

@ErikDeSmedt
Copy link
Contributor

Having a separate fee for on-chain is a good idea.

If we change the spec I'd like to consider how we could include dual funding as a payment option.
The proposal is final and has been merged a 2 weeks ago.

@SeverinAlexB SeverinAlexB added this to the Finalize LSPS1 milestone Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants