diff options
| -rw-r--r-- | lib/nwc.ts | 8 | ||||
| -rw-r--r-- | routes/payments.ts | 5 |
2 files changed, 12 insertions, 1 deletions
@@ -634,7 +634,13 @@ const checkBudget = async (app, amount) => { const handle = (method, params, ev, app, user) => ({ async pay_invoice() { - const { invoice: pr, metadata, amount: reqAmountMsat } = params; + const { metadata, amount: reqAmountMsat } = params; + // Lowercase so getInvoice() finds our own invoices below (see create() + // in routes/payments.ts). + const pr = + typeof params.invoice === "string" + ? params.invoice.replace(/\s/g, "").toLowerCase() + : params.invoice; const { amount_msat, payee } = await ln.decode(pr); const { id } = await ln.getinfo(); // For an amountless bolt11, the invoice carries no amount — NIP-47 lets the diff --git a/routes/payments.ts b/routes/payments.ts index 67243a12..66fd3386 100644 --- a/routes/payments.ts +++ b/routes/payments.ts @@ -66,6 +66,11 @@ export default { if (payreq != null && typeof payreq !== "string") fail("Invalid payment request"); if (hash != null && typeof hash !== "string") fail("Invalid invoice"); + // Invoices are keyed lowercase. An uppercase payreq (QR-scanned) missed + // the lookup below and was paid over cl to ourselves instead of + // internally; the listener then found the sender's payment:<bolt11> + // pointer and skipped crediting the recipient. + if (payreq) payreq = payreq.replace(/\s/g, "").toLowerCase(); if (typeof amount !== "undefined") { amount = Number.parseInt(amount); |