Skip to content

Commit

Permalink
Document ser for BOLT 12 pending outbound payments.
Browse files Browse the repository at this point in the history
Useful since these variants will be dropped on downgrade, so want some text to
explain why that's okay.
  • Loading branch information
valentinewallace committed Jul 26, 2024
1 parent 8b708e7 commit 60c390f
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions lightning/src/ln/outbound_payment.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2005,17 +2005,23 @@ impl_writeable_tlv_based_enum_upgradable!(PendingOutboundPayment,
(1, reason, option),
(2, payment_hash, required),
},
// Added in 0.0.123. Prior versions will drop these outbounds on downgrade, which is safe because
// no HTLCs are in-flight.
(5, AwaitingInvoice) => {
(0, expiration, required),
(1, invoice_request, option),
(2, retry_strategy, required),
(4, max_total_routing_fee_msat, option),
},
// Added in 0.0.123. Prior versions will drop these outbounds on downgrade, which is safe because
// no HTLCs are in-flight.
(7, InvoiceReceived) => {
(0, payment_hash, required),
(2, retry_strategy, required),
(4, max_total_routing_fee_msat, option),
},
// Added in 0.0.124. Prior versions will drop these outbounds on downgrade, which is safe because
// no HTLCs are in-flight.
(9, StaticInvoiceReceived) => {
(0, payment_hash, required),
(2, keysend_preimage, required),
Expand Down

0 comments on commit 60c390f

Please sign in to comment.