Skip to content

Commit e39310c

Browse files
committed
f expand top-level ChannelManager comments some
1 parent 202ad2f commit e39310c

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

lightning/src/ln/channelmanager.rs

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -577,13 +577,14 @@ pub type SimpleRefChannelManager<'a, 'b, 'c, 'd, 'e, 'f, 'g, 'h, M, T, F, L> = C
577577
/// offline for a full minute. In order to track this, you must call
578578
/// timer_tick_occurred roughly once per minute, though it doesn't have to be perfect.
579579
///
580-
/// ChannelManager limits the number of inbound connections and inbound channels which have not yet
581-
/// had their funding transaction confirmed. This may result in nodes which we do not have a
582-
/// channel with being unable to connect to us or open new channels with us if we have many peers
583-
/// with unfunded channels.
580+
/// To avoid trivial DoS issues, ChannelManager limits the number of inbound connections and
581+
/// inbound channels which have not yet had their funding transaction confirmed. This may result in
582+
/// nodes which we do not have a channel with being unable to connect to us or open new channels
583+
/// with us if we have many peers with unfunded channels.
584584
///
585585
/// Because it is an indication of trust, inbound channels which we've accepted as 0conf are
586-
/// exempted from the count of unfunded channels.
586+
/// exempted from the count of unfunded channels. Similarly, outbound channels and connections are
587+
/// never limited.
587588
///
588589
/// Rather than using a plain ChannelManager, it is preferable to use either a SimpleArcChannelManager
589590
/// a SimpleRefChannelManager, for conciseness. See their documentation for more details, but

0 commit comments

Comments
 (0)