Skip to content

Commit 1d0aec2

Browse files
committed
fixup! Start publishing PaymentForwarded events.
1 parent 7585934 commit 1d0aec2

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

src/event.rs

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ impl_writeable_tlv_based_enum!(Event,
259259
(7, PaymentForwarded) => {
260260
(0, prev_channel_id, required),
261261
(2, next_channel_id, required),
262-
(4, prev_user_channel_id, required),
262+
(4, prev_user_channel_id, option),
263263
(6, next_user_channel_id, option),
264264
(8, total_fee_earned_msat, option),
265265
(10, skimmed_fee_msat, option),
@@ -1139,20 +1139,17 @@ where
11391139
let event = Event::PaymentForwarded {
11401140
prev_channel_id: prev_channel_id.expect("prev_channel_id expected for events generated by LDK versions greater than 0.0.107."),
11411141
next_channel_id: next_channel_id.expect("next_channel_id expected for events generated by LDK versions greater than 0.0.107."),
1142-
prev_user_channel_id: prev_user_channel_id.map(|u|UserChannelId(u)),
1143-
next_user_channel_id: next_user_channel_id.map(|u| UserChannelId(u)),
1142+
prev_user_channel_id: prev_user_channel_id.map(UserChannelId),
1143+
next_user_channel_id: next_user_channel_id.map(UserChannelId),
11441144
total_fee_earned_msat,
11451145
skimmed_fee_msat,
11461146
claim_from_onchain_tx,
11471147
outbound_amount_forwarded_msat,
11481148
};
1149-
match self.event_queue.add_event(event) {
1150-
Ok(_) => {},
1151-
Err(e) => {
1152-
log_error!(self.logger, "Failed to push to event queue: {}", e);
1153-
return Err(ReplayEvent());
1154-
},
1155-
};
1149+
self.event_queue.add_event(event).map_err(|e| {
1150+
log_error!(self.logger, "Failed to push to event queue: {}", e);
1151+
ReplayEvent()
1152+
})?;
11561153

11571154
let read_only_network_graph = self.network_graph.read_only();
11581155
let nodes = read_only_network_graph.nodes();

tests/integration_tests_rust.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,9 @@ fn multi_hop_sending() {
192192
let invoice = nodes[4].bolt11_payment().receive(2_500_000, &"asdf", 9217).unwrap();
193193
nodes[0].bolt11_payment().send(&invoice, Some(sending_params)).unwrap();
194194

195+
expect_event!(nodes[1], PaymentForwarded);
196+
expect_event!(nodes[2], PaymentForwarded);
197+
195198
let payment_id = expect_payment_received_event!(&nodes[4], 2_500_000);
196199
let fee_paid_msat = Some(2000);
197200
expect_payment_successful_event!(nodes[0], payment_id, Some(fee_paid_msat));

0 commit comments

Comments
 (0)