keysend: Add maxfee to keysend for consistency with pay

Author: s373nZ

.msggen.json

@@ -1521,6 +1521,7 @@
             "KeySend.extratlvs": 9,
             "KeySend.label": 3,
             "KeySend.maxdelay": 6,
+            "KeySend.maxfee": 11,
             "KeySend.maxfeepercent": 4,
             "KeySend.msatoshi": 2,
             "KeySend.retry_for": 5,
@@ -6362,6 +6363,10 @@
             "added": "pre-v0.10.1",
             "deprecated": null
         },
+        "KeySend.maxfee": {
+            "added": "v24.11",
+            "deprecated": null
+        },
         "KeySend.maxfeepercent": {
             "added": "pre-v0.10.1",
             "deprecated": null

CHANGELOG.md

@@ -3,6 +3,11 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
+## [Unreleased]
+
+### Added
+
+  - keysend: Add `maxfee` to keysend for consistency with pay. ([#7227])
 
 ## [24.08] - 2024-08-28: "Steel Backed-up Channels"
 

cln-grpc/proto/node.proto

@@ -293,6 +293,7 @@ fn test_keysend() {
                 }],
             }),
             extratlvs: None,
+            maxfee: None
         };
 
     let u: cln_rpc::model::requests::KeysendRequest = g.into();

cln-grpc/src/convert.rs

@@ -15048,6 +15048,13 @@
             "description": [
               "Dictionary of additional fields to insert into the final tlv. The format is 'fieldnumber': 'hexstring'."
             ]
+          },
+          "maxfee": {
+            "added": "v24.11",
+            "type": "msat",
+            "description": [
+              "*maxfee* overrides both *maxfeepercent* and *exemptfee* defaults (and if you specify *maxfee* you cannot specify either of those), and creates an absolute limit on what fee we will pay. This allows you to implement your own heuristics rather than the primitive ones used here."
+            ]
           }
         }
       },

cln-grpc/src/test.rs

@@ -103,6 +103,13 @@
         "description": [
           "Dictionary of additional fields to insert into the final tlv. The format is 'fieldnumber': 'hexstring'."
         ]
+      },
+      "maxfee": {
+        "added": "v24.11",
+        "type": "msat",
+        "description": [
+          "*maxfee* overrides both *maxfeepercent* and *exemptfee* defaults (and if you specify *maxfee* you cannot specify either of those), and creates an absolute limit on what fee we will pay. This allows you to implement your own heuristics rather than the primitive ones used here."
+        ]
       }
     }
   },

cln-rpc/src/model.rs

@@ -23,7 +23,7 @@ static u64 *accepted_extra_tlvs;
  * ================
  *
  * The keysend modifier adds the payment preimage to the TLV payload. This
- * enables the recipient to accept the payment despite it not correspondin to
+ * enables the recipient to accept the payment despite it not corresponding to
  * an invoice that the recipient created. Keysend does not provide any proof
  * or payment, but does not require an out-of-band communication round to get
  * an invoice first.
@@ -215,7 +215,7 @@ static struct command_result *json_keysend(struct command *cmd, const char *buf,
 {
 	struct payment *p;
 	const char *label;
-	struct amount_msat *exemptfee, *msat;
+	struct amount_msat *exemptfee, *msat, *maxfee;
 	struct node_id *destination;
 	u64 *maxfee_pct_millionths;
 	u32 *maxdelay;
@@ -229,14 +229,15 @@ static struct command_result *json_keysend(struct command *cmd, const char *buf,
 		   p_req("destination", param_node_id, &destination),
 		   p_req("amount_msat", param_msat, &msat),
 		   p_opt("label", param_string, &label),
-		   p_opt_def("maxfeepercent", param_millionths,
-			     &maxfee_pct_millionths, 500000),
+		   p_opt("maxfeepercent", param_millionths,
+			 &maxfee_pct_millionths),
 		   p_opt_def("retry_for", param_number, &retryfor, 60),
 		   p_opt_def("maxdelay", param_number, &maxdelay,
 			     maxdelay_default),
-		   p_opt_def("exemptfee", param_msat, &exemptfee, AMOUNT_MSAT(5000)),
+		   p_opt("exemptfee", param_msat, &exemptfee),
 		   p_opt("extratlvs", param_extra_tlvs, &extra_fields),
 		   p_opt("routehints", param_routehint_array, &hints),
+		   p_opt("maxfee", param_msat, &maxfee),
 		   p_opt_dev("dev_use_shadow", param_bool, &dev_use_shadow, true),
 		   NULL))
 		return command_param_failed();
@@ -272,19 +273,36 @@ static struct command_result *json_keysend(struct command *cmd, const char *buf,
 		    "We are the destination. Keysend cannot be used to send funds to yourself");
 	}
 
-	if (!amount_msat_fee(&p->constraints.fee_budget, p->our_amount, 0,
-			     *maxfee_pct_millionths / 100)) {
-		return command_fail(
-		    cmd, JSONRPC2_INVALID_PARAMS,
-		    "Overflow when computing fee budget, fee rate too high.");
-	}
-
 	p->constraints.cltv_budget = *maxdelay;
 
 	payment_mod_keysend_get_data(p)->extra_tlvs =
 	    tal_steal(p, extra_fields);
 
-	payment_mod_exemptfee_get_data(p)->amount = *exemptfee;
+	if (maxfee) {
+		if (maxfee_pct_millionths || exemptfee) {
+			return command_fail(
+				cmd, JSONRPC2_INVALID_PARAMS,
+				"If you specify maxfee, cannot specify maxfeepercent or exemptfee.");
+		}
+		p->constraints.fee_budget = *maxfee;
+		payment_mod_exemptfee_get_data(p)->amount = AMOUNT_MSAT(0);
+	} else {
+		u64 maxppm;
+
+		if (maxfee_pct_millionths)
+			maxppm = *maxfee_pct_millionths / 100;
+		else
+			maxppm = 500000 / 100;
+		if (!amount_msat_fee(&p->constraints.fee_budget, p->our_amount, 0,
+				     maxppm)) {
+			return command_fail(
+				cmd, JSONRPC2_INVALID_PARAMS,
+				"Overflow when computing fee budget, fee rate too high.");
+		}
+		payment_mod_exemptfee_get_data(p)->amount
+			= exemptfee ? *exemptfee : AMOUNT_MSAT(5000);
+	}
+
 	payment_mod_shadowroute_get_data(p)->use_shadow = *dev_use_shadow;
 	p->label = tal_steal(p, label);