Skip to content

Commit f6120b8

Browse files
committed
Revert "Retransmit channel_ready / splice_locked awaiting announcement_sigs"
This reverts commit 48ddcc61975580579e056b9087ace28c5c887b61.
1 parent e7fe6d0 commit f6120b8

File tree

1 file changed

+0
-57
lines changed

1 file changed

+0
-57
lines changed

lightning/src/ln/channel.rs

Lines changed: 0 additions & 57 deletions
Original file line numberDiff line numberDiff line change
@@ -8793,65 +8793,8 @@ where
87938793
let channel_ready = if msg.next_local_commitment_number == 1 && INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.transaction_number() == 1 {
87948794
// We should never have to worry about MonitorUpdateInProgress resending ChannelReady
87958795
self.get_channel_ready(logger)
8796-
} else if splicing_negotiated {
8797-
// A node:
8798-
// - if `option_splice` was negotiated and `your_last_funding_locked` is not
8799-
// set in the `channel_reestablish` it received:
8800-
// - MUST retransmit `channel_ready`.
8801-
msg.your_last_funding_locked_txid
8802-
.is_none()
8803-
.then(|| ())
8804-
// The sending node:
8805-
// - if `my_current_funding_locked` is included:
8806-
// - if `announce_channel` is set for this channel:
8807-
// - if it has not received `announcement_signatures` for that transaction:
8808-
// - MUST retransmit `channel_ready` or `splice_locked` after exchanging `channel_reestablish`.
8809-
.or_else(|| {
8810-
self.maybe_get_my_current_funding_locked()
8811-
.filter(|funding| !funding.is_splice())
8812-
.filter(|_| self.context.config.announce_for_forwarding)
8813-
.filter(|_| self.context.announcement_sigs.is_none())
8814-
.map(|_| ())
8815-
})
8816-
.and_then(|_| self.get_channel_ready(logger))
88178796
} else { None };
88188797

8819-
let sent_splice_txid = self
8820-
.maybe_get_my_current_funding_locked()
8821-
.filter(|funding| funding.is_splice())
8822-
.map(|funding| {
8823-
funding.get_funding_txid().expect("Splice funding_txid should always be set")
8824-
});
8825-
let splice_locked = msg
8826-
// A receiving node:
8827-
// - if `your_last_funding_locked` is set and it does not match the most recent
8828-
// `splice_locked` it has sent:
8829-
// - MUST retransmit `splice_locked`.
8830-
.your_last_funding_locked_txid
8831-
.and_then(|last_funding_txid| {
8832-
sent_splice_txid.filter(|sent_splice_txid| last_funding_txid != *sent_splice_txid)
8833-
})
8834-
// The sending node:
8835-
// - if `my_current_funding_locked` is included:
8836-
// - if `announce_channel` is set for this channel:
8837-
// - if it has not received `announcement_signatures` for that transaction:
8838-
// - MUST retransmit `channel_ready` or `splice_locked` after exchanging `channel_reestablish`.
8839-
.or_else(|| {
8840-
sent_splice_txid
8841-
.filter(|_| self.context.config.announce_for_forwarding)
8842-
.filter(|sent_splice_txid| {
8843-
if self.funding.get_funding_txid() == Some(*sent_splice_txid) {
8844-
self.context.announcement_sigs.is_none()
8845-
} else {
8846-
true
8847-
}
8848-
})
8849-
})
8850-
.map(|splice_txid| msgs::SpliceLocked {
8851-
channel_id: self.context.channel_id,
8852-
splice_txid,
8853-
});
8854-
88558798
let mut commitment_update = None;
88568799
let mut tx_signatures = None;
88578800
let mut tx_abort = None;

0 commit comments

Comments
 (0)