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

lndx #1

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

lndx #1

wants to merge 8 commits into from

Conversation

doitian
Copy link
Owner

@doitian doitian commented Mar 19, 2024

Demo

See the docs: https://github.com/doitian/lnd/blob/lndx/docs/lndx.md

Changes Map for Review

  • witness_beacon.go: Allow subscribe to any preimage received from settled htlcs.
  • rpcserver.go: For the new lncli command addinvoiceproxy.
  • invoiceregistry.go: When the node accepts a hodl invoice, and if it is a proxy invoice, fullfil the original payment request.
  • server.go: The implementation to get the preimage by fulfilling the original payment request.

Known issues

The proxy invoice will be stuck if the node restarts.

Actions

  • lncli: add command addinvoiceproxy
  • lnrpc: add skeleton for rpc AddInvoiceProxy
  • lnrpc: allows inserting a hold invoice which has no preimage
  • invoicedb: add a column to save the original payment request
  • payment: get preimage by fullfilling the original payment request

@doitian doitian changed the base branch from base to master March 20, 2024 07:47
@doitian doitian force-pushed the lndx branch 2 times, most recently from e915f75 to 12a15dc Compare March 25, 2024 07:00
Get the preimage by fullfilling the original payment request.
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

Successfully merging this pull request may close these issues.

1 participant