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

Remove auto-settle mode #100

Open
joostjager opened this issue Oct 21, 2022 · 1 comment
Open

Remove auto-settle mode #100

joostjager opened this issue Oct 21, 2022 · 1 comment

Comments

@joostjager
Copy link
Contributor

This issue serves as a place for discussing whether we should remove AutoSettle. It requires additional code paths, and arguably the added value is only in testing/development. The intended user group for lnmux is unlikely to need this functionality.

Alternatively it can be implemented in a script that subscribes to all payments via SubscribeInvoiceAccepted and automatically calls SettleInvoice for each of them.

@Torakushi
Copy link
Contributor

Interesting idea!

Wouldn’t it useful for people that will use lnmux as a virtual node? (for inbound fees for example)
Maybe they won't want to rework their existing architecture to subscribe to lnmux events, but just use it as a virtual node

But for that, it is necessary to have a RPC call to retrieve all paid invoices (as there will be no settlement notifications).
(see this issue)

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

2 participants