From 599eb2ee993d42a8eae88c75d53347db02838950 Mon Sep 17 00:00:00 2001 From: Olaoluwa Osuntokun Date: Tue, 4 Jul 2023 17:21:46 -0700 Subject: [PATCH] challenger: skip trying to load+parse AMP invoices AMP invoiecs don't have a payment hash, as it's based on payment addr and set id. --- challenger/lnd.go | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/challenger/lnd.go b/challenger/lnd.go index 9e10a02..00bd53e 100644 --- a/challenger/lnd.go +++ b/challenger/lnd.go @@ -102,6 +102,12 @@ func (l *LndChallenger) Start() error { // updates for new invoices and/or newly settled invoices. l.invoicesMtx.Lock() for _, invoice := range invoiceResp.Invoices { + // Some invoices like AMP invoices may not have a payment hash + // populated. + if invoice.RHash == nil { + continue + } + if invoice.AddIndex > addIndex { addIndex = invoice.AddIndex } @@ -207,6 +213,12 @@ func (l *LndChallenger) readInvoiceStream( default: } + // Some invoices like AMP invoices may not have a payment hash + // populated. + if invoice.RHash == nil { + continue + } + hash, err := lntypes.MakeHash(invoice.RHash) if err != nil { log.Errorf("Error parsing invoice hash: %v", err)