Doc that `Record::node_id` may be missing even with `channel_id` 2023-12-2314-cleanups-3
authorMatt Corallo <git@bluematt.me>
Sun, 3 Dec 2023 19:09:32 +0000 (19:09 +0000)
committerMatt Corallo <git@bluematt.me>
Sun, 3 Dec 2023 19:42:21 +0000 (19:42 +0000)
There are various place where we log something related to a channel
but fail to fill in the channel's counterparty information. This is
somewhat surprising, given channel counterparty information is
always known, but simply is sometimes not readily accessible to LDK
when a log is printed.

lightning/src/util/logger.rs

index f1534933792949a41318c3b80e0cc1847cbf63f4..8ca5333f63daabf21a5baed562ec8d0ae2053327 100644 (file)
@@ -98,6 +98,10 @@ pub struct Record<'a> {
        /// The verbosity level of the message.
        pub level: Level,
        /// The node id of the peer pertaining to the logged record.
+       ///
+       /// Note that in some cases a [`Self::channel_id`] may be filled in but this may still be
+       /// `None`, depending on if the peer information is readily available in LDK when the log is
+       /// generated.
        pub peer_id: Option<PublicKey>,
        /// The channel id of the channel pertaining to the logged record. May be a temporary id before
        /// the channel has been funded.