offers: honor payment-fronting-nodes when creating invoice_requests.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

Author: rustyrussell

plugins/offers_offer.c

@@ -650,6 +650,12 @@ static struct command_result *found_best_peer_invrequest(struct command *cmd,
 	const struct offers_data *od = get_offers_data(cmd->plugin);
 
 	if (!best) {
+		/* Don't allow bare invoices if they explicitly told us to front */
+		if (od->fronting_nodes) {
+			return command_fail(cmd, LIGHTNINGD,
+					    "Could not find neighbour fronting node");
+		}
+
 		/* FIXME: Make this a warning in the result! */
 		plugin_log(cmd->plugin, LOG_UNUSUAL,
 			   "No incoming channel to public peer, so no blinded path for invoice request");
@@ -773,14 +779,12 @@ struct command_result *json_invoicerequest(struct command *cmd,
 	 *   - MUST set `invreq_features`.`features` to the bitmap of features.
 	 */
 
-	/* FIXME: We only set blinded path if private/noaddr, we should allow
-	 * setting otherwise! */
 	if (we_want_blinded_path(cmd->plugin, false)) {
 		struct invrequest_data *idata = tal(cmd, struct invrequest_data);
 		idata->invreq = invreq;
 		idata->single_use = *single_use;
 		idata->label = label;
-		return find_best_peer(cmd, OPT_ONION_MESSAGES, NULL,
+		return find_best_peer(cmd, OPT_ONION_MESSAGES, od->fronting_nodes,
 				      found_best_peer_invrequest, idata);
 	}
 

tests/test_invoices.py

@@ -979,3 +979,12 @@ def test_payment_fronting(node_factory):
 
     l1.rpc.xpay(l3invb12)
     l1.rpc.xpay(l4invb12)    
+
+    # Balance so l3 can pay ->l1->l4.
+    l3inv2 = l3.rpc.invoice(10000000, 'l3inv2', 'l3inv2')['bolt11']
+    l1.rpc.xpay(l3inv2)
+
+    # When l3 creates an invoice request, it will also use the fronting nodes.
+    l3invreq = l3.rpc.invoicerequest(amount=1000, description='l3invreq')['bolt12']
+    assert only_one(l3.rpc.decode(l3invreq)['invreq_paths'])['first_node_id'] == l1.info['id']
+    l4.rpc.sendinvoice(invreq=l3invreq, label='l3invreq')