Adds the ability to set a description or description hash for create_invoice which is used in LNv2
Implements create_invoice for CLN
This PR is unfortunately difficult to test. That is because create_invoice is only used for LNv2 currently add there aren't currently tests that exercise paying into a Fedimint from an external Lightning node. Adding support for generating the invoice for LNv1 is a longer discussion, since we would need to figure out what backwards compatibility support we want (IMO we might want to do this in the future depending on when LNv2 ships). Adding testing support for paying fedimints from an external wallet using LNv2 requires adding CLI commands, which is out of scope for this PR.
Since this only affect LNv2 currently, I hope we can merge and iterate if any problems are found while we add tests. I think we need to overhaul and refactor a lot of the Lightning tests https://github.com/fedimint/fedimint/issues/5168
create_invoicewhich is used in LNv2create_invoicefor CLNThis PR is unfortunately difficult to test. That is because
create_invoiceis only used for LNv2 currently add there aren't currently tests that exercise paying into a Fedimint from an external Lightning node. Adding support for generating the invoice for LNv1 is a longer discussion, since we would need to figure out what backwards compatibility support we want (IMO we might want to do this in the future depending on when LNv2 ships). Adding testing support for paying fedimints from an external wallet using LNv2 requires adding CLI commands, which is out of scope for this PR.Since this only affect LNv2 currently, I hope we can merge and iterate if any problems are found while we add tests. I think we need to overhaul and refactor a lot of the Lightning tests https://github.com/fedimint/fedimint/issues/5168