Skip to content

Commit 683e849

Browse files
committed
Correct ChannelManager in_flight_monitor_updates serialization
In e8854f9 we changed the type of `ChannelManager::in_flight_monitor_updates`, writing a legacy version and reading the new version as a new TLV field. Sadly, we spuriously marked the new TLV as `required`, breaking upgrade from 0.1. Here we fix the oversight by simply marking it `option`al.
1 parent 98297a6 commit 683e849

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lightning/src/ln/channelmanager.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13566,7 +13566,7 @@ where
1356613566
(13, htlc_onion_fields, optional_vec),
1356713567
(14, decode_update_add_htlcs_opt, option),
1356813568
(15, self.inbound_payment_id_secret, required),
13569-
(17, in_flight_monitor_updates, required),
13569+
(17, in_flight_monitor_updates, option),
1357013570
(19, peer_storage_dir, optional_vec),
1357113571
});
1357213572

@@ -14097,7 +14097,7 @@ where
1409714097
(13, claimable_htlc_onion_fields, optional_vec),
1409814098
(14, decode_update_add_htlcs, option),
1409914099
(15, inbound_payment_id_secret, option),
14100-
(17, in_flight_monitor_updates, required),
14100+
(17, in_flight_monitor_updates, option),
1410114101
(19, peer_storage_dir, optional_vec),
1410214102
});
1410314103
let mut decode_update_add_htlcs = decode_update_add_htlcs.unwrap_or_else(|| new_hash_map());

0 commit comments

Comments
 (0)