Skip to content

Commit 90c9763

Browse files
committed
Add logging in route hint filtering
Knowing why a channel was not included as an invoice route hint can be valuable. Add logging on the decisions made when filtering channels.
1 parent 92a1e49 commit 90c9763

File tree

4 files changed

+143
-70
lines changed

4 files changed

+143
-70
lines changed

lightning-invoice/src/payment.rs

+6-6
Original file line numberDiff line numberDiff line change
@@ -2199,8 +2199,8 @@ mod tests {
21992199
let invoice_payer = InvoicePayer::new(nodes[0].node, router, nodes[0].logger, event_handler, Retry::Attempts(1));
22002200

22012201
assert!(invoice_payer.pay_invoice(&create_invoice_from_channelmanager_and_duration_since_epoch(
2202-
&nodes[1].node, nodes[1].keys_manager, Currency::Bitcoin, Some(100_010_000), "Invoice".to_string(),
2203-
duration_since_epoch(), 3600).unwrap())
2202+
&nodes[1].node, nodes[1].keys_manager, nodes[1].logger, Currency::Bitcoin,
2203+
Some(100_010_000), "Invoice".to_string(), duration_since_epoch(), 3600).unwrap())
22042204
.is_ok());
22052205
let htlc_msgs = nodes[0].node.get_and_clear_pending_msg_events();
22062206
assert_eq!(htlc_msgs.len(), 2);
@@ -2244,8 +2244,8 @@ mod tests {
22442244
let invoice_payer = InvoicePayer::new(nodes[0].node, router, nodes[0].logger, event_handler, Retry::Attempts(1));
22452245

22462246
assert!(invoice_payer.pay_invoice(&create_invoice_from_channelmanager_and_duration_since_epoch(
2247-
&nodes[1].node, nodes[1].keys_manager, Currency::Bitcoin, Some(100_010_000), "Invoice".to_string(),
2248-
duration_since_epoch(), 3600).unwrap())
2247+
&nodes[1].node, nodes[1].keys_manager, nodes[1].logger, Currency::Bitcoin,
2248+
Some(100_010_000), "Invoice".to_string(), duration_since_epoch(), 3600).unwrap())
22492249
.is_ok());
22502250
let htlc_msgs = nodes[0].node.get_and_clear_pending_msg_events();
22512251
assert_eq!(htlc_msgs.len(), 2);
@@ -2325,8 +2325,8 @@ mod tests {
23252325
let invoice_payer = InvoicePayer::new(nodes[0].node, router, nodes[0].logger, event_handler, Retry::Attempts(1));
23262326

23272327
assert!(invoice_payer.pay_invoice(&create_invoice_from_channelmanager_and_duration_since_epoch(
2328-
&nodes[1].node, nodes[1].keys_manager, Currency::Bitcoin, Some(100_010_000), "Invoice".to_string(),
2329-
duration_since_epoch(), 3600).unwrap())
2328+
&nodes[1].node, nodes[1].keys_manager, nodes[1].logger, Currency::Bitcoin,
2329+
Some(100_010_000), "Invoice".to_string(), duration_since_epoch(), 3600).unwrap())
23302330
.is_ok());
23312331
let htlc_updates = SendEvent::from_node(&nodes[0]);
23322332
check_added_monitors!(nodes[0], 1);

0 commit comments

Comments
 (0)