f - expand docs

Author: jkczyz

lightning/src/ln/channelmanager.rs

@@ -2292,6 +2292,11 @@ const MAX_NO_CHANNEL_PEERS: usize = 250;
 /// will included a [`BlindedPath`] created using:
 /// - [`MessageRouter::create_compact_blinded_paths`] when short-lived, and
 /// - [`MessageRouter::create_blinded_paths`] when long-lived.
+///
+/// Using compact [`BlindedPath`]s may provide better privacy as more hops can be used with the same
+/// amount of bytes. However, since they use short channel ids instead of pubkeys, they are more
+/// likely to become invalid over time as channels are closed. Thus, they are only suitable for
+/// short-term use.
 pub const MAX_SHORT_LIVED_RELATIVE_EXPIRY: Duration = Duration::from_secs(60 * 60 * 24);
 
 /// Information needed for constructing an invoice route hint for this channel.
@@ -8558,10 +8563,11 @@ macro_rules! create_offer_builder { ($self: ident, $builder: ty) => {
 	/// # Privacy
 	///
 	/// Uses [`MessageRouter`] to construct a [`BlindedPath`] for the offer based on the given
-	/// `absolute_expiry` according to [`MAX_SHORT_LIVED_RELATIVE_EXPIRY`]. However, if one is not
-	/// found, uses a one-hop [`BlindedPath`] with [`ChannelManager::get_our_node_id`] as the
-	/// introduction node instead. In the latter case, the node must be announced, otherwise, there
-	/// is no way to find a path to the introduction in order to send the [`InvoiceRequest`].
+	/// `absolute_expiry` according to [`MAX_SHORT_LIVED_RELATIVE_EXPIRY`]. See those docs for
+	/// privacy implications. However, if one is not found, uses a one-hop [`BlindedPath`] with
+	/// [`ChannelManager::get_our_node_id`] as the introduction node instead. In the latter case,
+	/// the node must be announced, otherwise, there is no way to find a path to the introduction
+	/// node in order to send the [`InvoiceRequest`].
 	///
 	/// Also, uses a derived signing pubkey in the offer for recipient privacy.
 	///
@@ -8625,10 +8631,11 @@ macro_rules! create_refund_builder { ($self: ident, $builder: ty) => {
 	/// # Privacy
 	///
 	/// Uses [`MessageRouter`] to construct a [`BlindedPath`] for the refund based on the given
-	/// `absolute_expiry` according to [`MAX_SHORT_LIVED_RELATIVE_EXPIRY`]. However, if one is not
-	/// found, uses a one-hop [`BlindedPath`] with [`ChannelManager::get_our_node_id`] as the
-	/// introduction node instead. In the latter case, the node must be announced, otherwise, there
-	/// is no way to find a path to the introduction in order to send the [`Bolt12Invoice`].
+	/// `absolute_expiry` according to [`MAX_SHORT_LIVED_RELATIVE_EXPIRY`]. See those docs for
+	/// privacy implications. However, if one is not found, uses a one-hop [`BlindedPath`] with
+	/// [`ChannelManager::get_our_node_id`] as the introduction node instead. In the latter case,
+	/// the node must be announced, otherwise, there is no way to find a path to the introduction
+	/// node in order to send the [`Bolt12Invoice`].
 	///
 	/// Also, uses a derived payer id in the refund for payer privacy.
 	///