Merge pull request #8823 from bhandras/fixup-sendpayment-amp

lncli: fix parsing of --amp when sending a payment
This commit is contained in:
Oliver Gugger 2024-06-10 10:23:45 -06:00 committed by GitHub
commit 3549d719ee
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 7 additions and 2 deletions

View File

@ -328,14 +328,15 @@ func sendPayment(ctx *cli.Context) error {
PaymentRequest: stripPrefix(ctx.String("pay_req")),
Amt: ctx.Int64("amt"),
DestCustomRecords: make(map[uint64][]byte),
Amp: ctx.Bool(ampFlag.Name),
}
// We'll attempt to parse a payment address as well, given that
// if the user is using an AMP invoice, then they may be trying
// to specify that value manually.
//
// Don't parse unnamed arguments to prevent confusion with the main
// unnamed argument format for non-AMP payments.
// Don't parse unnamed arguments to prevent confusion with the
// main unnamed argument format for non-AMP payments.
payAddr, err := parsePayAddr(ctx, nil)
if err != nil {
return err

View File

@ -46,6 +46,9 @@
* [`getchaninfo`](https://github.com/lightningnetwork/lnd/pull/8813) now accepts
a channel outpoint besides a channel id.
* [Fixed](https://github.com/lightningnetwork/lnd/pull/8823) how we parse the
`--amp` flag when sending a payment specifying the payment request.
## Code Health
## Breaking Changes
## Performance Improvements
@ -59,4 +62,5 @@
# Contributors (Alphabetical Order)
* Andras Banki-Horvath
* Bufo