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

RPC: Document "payment_addr" usage #4839

Closed
C-Otto opened this issue Dec 7, 2020 · 1 comment
Closed

RPC: Document "payment_addr" usage #4839

C-Otto opened this issue Dec 7, 2020 · 1 comment

Comments

@C-Otto
Copy link
Contributor

C-Otto commented Dec 7, 2020

The protocol documentation includes a field payment_addr for AddInvoice:
https://api.lightning.community/#addinvoice

The documentation reads

The payment address of this invoice. This value will be used in MPP payments, and also for newer invoies [sic] that always require the MPP paylaod [sic] for added end-to-end security.

While this might explain the need for the field, it does not tell me how to populate the field when I create a new invoice. The code seems to generate some random bytes for this (see https://github.com/lightningnetwork/lnd/blob/master/lnrpc/invoicesrpc/addinvoice.go#L313), and the data provided via the RPC call seems to be ignored (see https://github.com/lightningnetwork/lnd/blob/master/rpcserver.go#L4739).

Please fix the documentation so that RPC users have an idea what kind of data to provide there.

Related:

Your environment

  • version of lnd: commit=v0.11.0-beta-509-g7f9f4a7c8d46962b9f4a6e2fbec716c6511fdd97
  • which operating system (uname -a on *Nix): Linux ftp 5.8.0-0.bpo.2-amd64 Fix name typo in README #1 SMP Debian 5.8.10-1~bpo10+1 (2020-09-26) x86_64 GNU/Linux
  • version of btcd, bitcoind, or other backend: Bitcoin Core 0.20.0
@C-Otto
Copy link
Contributor Author

C-Otto commented Jan 3, 2021

Superseeded by #4893 (as the "payment_addr" field is not supposed to be an input to the add invoice call).

@C-Otto C-Otto closed this as completed Jan 3, 2021
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

1 participant