lnd icon indicating copy to clipboard operation
lnd copied to clipboard

lncli+lnrpc: add hints flag to addinvoice command

Open hotorb-code opened this issue 2 years ago • 7 comments

Change Description

This PR introduces the new flag hints for the addinvoice command. hints sets the number of routing hints the invoice should include. Poor channel liquidity may result in more hints than specified being added, lack of private channels in less. The maximum number of hints stays at 20. The private flag behavior is unchanged and keeps its focus on privacy. The private, hints, amt and amt_msat flags can be used together or separately depending on the desired result.

Fixes #7305.

Steps to Test

Create a private invoice with enough private channels.

Pull Request Checklist

Testing

  • [ ] Your PR passes all CI checks.
  • [ ] Tests covering the positive and negative (error paths) are included.
  • [ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

📝 Please see our Contribution Guidelines for further guidance.

hotorb-code avatar Mar 08 '23 02:03 hotorb-code

Thanks for the reviews. This is turning into a different PR so I have edited the title and description. The new approach is to add a hints flag to the addinvoice command.

hotorb-code avatar Apr 17 '23 14:04 hotorb-code

What is the correct way to compile the proto files? Is there a Docker image I should use? I'm following the readmes and have Docker and Go installed. It seems to compile and format in Docker but even the source gives me formatting errors. Here's what I do: git clone https://github.com/lightningnetwork/lnd cd lnd make rpc-check Errors with "Protos not properly formatted or not compiled with v3.4.0".

hotorb-code avatar May 01 '23 17:05 hotorb-code

You need to run make rpc to compile them. The make rpc-check is for the CI to check that the protos were compiled.

guggero avatar May 01 '23 20:05 guggero

Is this still under development? cc @hotorb-code

yyforyongyu avatar Jun 06 '23 06:06 yyforyongyu

I'm working on this again and made some changes.

hotorb-code avatar Oct 31 '23 17:10 hotorb-code

is this still active? if so could you do a rebase? thanks!

yyforyongyu avatar Apr 24 '24 17:04 yyforyongyu

@positiveblue: review reminder @yyforyongyu: review reminder

lightninglabs-deploy avatar May 22 '24 19:05 lightninglabs-deploy

@positiveblue: review reminder @yyforyongyu: review reminder

lightninglabs-deploy avatar Jun 12 '24 21:06 lightninglabs-deploy

!lightninglabs-deploy mute

yyforyongyu avatar Jun 13 '24 01:06 yyforyongyu

replaced by: https://github.com/lightningnetwork/lnd/pull/8819

EDIT: well, at least the cmd part of it. So if you want to continue with this, @hotorb-code, then perhaps can just update it to just do the "NumHopHints" part

ellemouton avatar Jun 13 '24 14:06 ellemouton