github.com/badrootd/nibiru-cometbft@v0.37.5-0.20240307173500-2a75559eee9b/docs/architecture/adr-061-p2p-refactor-scope.md (about)

     1  # ADR 061: P2P Refactor Scope
     2  
     3  ## Changelog
     4  
     5  - 2020-10-30: Initial version (@erikgrinaker)
     6  
     7  ## Context
     8  
     9  The `p2p` package responsible for peer-to-peer networking is rather old and has a number of weaknesses, including tight coupling, leaky abstractions, lack of tests, DoS vulnerabilites, poor performance, custom protocols, and incorrect behavior. A refactor has been discussed for several years ([#2067](https://github.com/tendermint/tendermint/issues/2067)).
    10  
    11  Informal Systems are also building a Rust implementation of Tendermint, [Tendermint-rs](https://github.com/informalsystems/tendermint-rs), and plan to implement P2P networking support over the next year. As part of this work, they have requested adopting e.g. [QUIC](https://datatracker.ietf.org/doc/draft-ietf-quic-transport/) as a transport protocol instead of implementing the custom application-level `MConnection` stream multiplexing protocol that Tendermint currently uses.
    12  
    13  This ADR summarizes recent discussion with stakeholders on the scope of a P2P refactor. Specific designs and implementations will be submitted as separate ADRs.
    14  
    15  ## Alternative Approaches
    16  
    17  There have been recurring proposals to adopt [LibP2P](https://libp2p.io) instead of maintaining our own P2P networking stack (see [#3696](https://github.com/tendermint/tendermint/issues/3696)). While this appears to be a good idea in principle, it would be a highly breaking protocol change, there are indications that we might have to fork and modify LibP2P, and there are concerns about the abstractions used.
    18  
    19  In discussions with Informal Systems we decided to begin with incremental improvements to the current P2P stack, add support for pluggable transports, and then gradually start experimenting with LibP2P as a transport layer. If this proves successful, we can consider adopting it for higher-level components at a later time.
    20  
    21  ## Decision
    22  
    23  The P2P stack will be refactored and improved iteratively, in several phases:
    24  
    25  * **Phase 1:** code and API refactoring, maintaining protocol compatibility as far as possible.
    26  
    27  * **Phase 2:** additional transports and incremental protocol improvements.
    28  
    29  * **Phase 3:** disruptive protocol changes.
    30  
    31  The scope of phases 2 and 3 is still uncertain, and will be revisited once the preceding phases have been completed as we'll have a better sense of requirements and challenges.
    32  
    33  ## Detailed Design
    34  
    35  Separate ADRs will be submitted for specific designs and changes in each phase, following research and prototyping. Below are objectives in order of priority.
    36  
    37  ### Phase 1: Code and API Refactoring
    38  
    39  This phase will focus on improving the internal abstractions and implementations in the `p2p` package. As far as possible, it should not change the P2P protocol in a backwards-incompatible way.
    40  
    41  * Cleaner, decoupled abstractions for e.g. `Reactor`, `Switch`, and `Peer`. [#2067](https://github.com/tendermint/tendermint/issues/2067) [#5287](https://github.com/tendermint/tendermint/issues/5287) [#3833](https://github.com/tendermint/tendermint/issues/3833)
    42      * Reactors should receive messages in separate goroutines or via buffered channels. [#2888](https://github.com/tendermint/tendermint/issues/2888)
    43  * Improved peer lifecycle management. [#3679](https://github.com/tendermint/tendermint/issues/3679) [#3719](https://github.com/tendermint/tendermint/issues/3719) [#3653](https://github.com/tendermint/tendermint/issues/3653) [#3540](https://github.com/tendermint/tendermint/issues/3540) [#3183](https://github.com/tendermint/tendermint/issues/3183) [#3081](https://github.com/tendermint/tendermint/issues/3081) [#1356](https://github.com/tendermint/tendermint/issues/1356)
    44      * Peer prioritization. [#2860](https://github.com/tendermint/tendermint/issues/2860) [#2041](https://github.com/tendermint/tendermint/issues/2041)
    45  * Pluggable transports, with `MConnection` as one implementation. [#5587](https://github.com/tendermint/tendermint/issues/5587) [#2430](https://github.com/tendermint/tendermint/issues/2430) [#805](https://github.com/tendermint/tendermint/issues/805)
    46  * Improved peer address handling.
    47      * Address book refactor. [#4848](https://github.com/tendermint/tendermint/issues/4848) [#2661](https://github.com/tendermint/tendermint/issues/2661)
    48      * Transport-agnostic peer addressing. [#5587](https://github.com/tendermint/tendermint/issues/5587) [#3782](https://github.com/tendermint/tendermint/issues/3782) [#3692](https://github.com/tendermint/tendermint/issues/3692)
    49      * Improved detection and advertisement of own address. [#5588](https://github.com/tendermint/tendermint/issues/5588) [#4260](https://github.com/tendermint/tendermint/issues/4260) [#3716](https://github.com/tendermint/tendermint/issues/3716) [#1727](https://github.com/tendermint/tendermint/issues/1727)
    50      * Support multiple IPs per peer. [#1521](https://github.com/tendermint/tendermint/issues/1521) [#2317](https://github.com/tendermint/tendermint/issues/2317)
    51  
    52  The refactor should attempt to address the following secondary objectives: testability, observability, performance, security, quality-of-service, backpressure, and DoS resilience. Much of this will be revisited as explicit objectives in phase 2.
    53  
    54  Ideally, the refactor should happen incrementally, with regular merges to `master` every few weeks. This will take more time overall, and cause frequent breaking changes to internal Go APIs, but it reduces the branch drift and gets the code tested sooner and more broadly.
    55  
    56  ### Phase 2: Additional Transports and Protocol Improvements
    57  
    58  This phase will focus on protocol improvements and other breaking changes. The following are considered proposals that will need to be evaluated separately once the refactor is done. Additional proposals are likely to be added during phase 1.
    59  
    60  * QUIC transport. [#198](https://github.com/tendermint/spec/issues/198)
    61  * Noise protocol for secret connection handshake. [#5589](https://github.com/tendermint/tendermint/issues/5589) [#3340](https://github.com/tendermint/tendermint/issues/3340)
    62  * Peer ID in connection handshake. [#5590](https://github.com/tendermint/tendermint/issues/5590)
    63  * Peer and service discovery (e.g. RPC nodes, state sync snapshots). [#5481](https://github.com/tendermint/tendermint/issues/5481) [#4583](https://github.com/tendermint/tendermint/issues/4583)
    64  * Rate-limiting, backpressure, and QoS scheduling. [#4753](https://github.com/tendermint/tendermint/issues/4753) [#2338](https://github.com/tendermint/tendermint/issues/2338)
    65  * Compression. [#2375](https://github.com/tendermint/tendermint/issues/2375)
    66  * Improved metrics and tracing. [#3849](https://github.com/tendermint/tendermint/issues/3849) [#2600](https://github.com/tendermint/tendermint/issues/2600)
    67  * Simplified P2P configuration options.
    68  
    69  ### Phase 3: Disruptive Protocol Changes
    70  
    71  This phase covers speculative, wide-reaching proposals that are poorly defined and highly uncertain. They will be evaluated once the previous phases are done.
    72  
    73  * Adopt LibP2P. [#3696](https://github.com/tendermint/tendermint/issues/3696)
    74  * Allow cross-reactor communication, possibly without channels.
    75  * Dynamic channel advertisment, as reactors are enabled/disabled. [#4394](https://github.com/tendermint/tendermint/issues/4394) [#1148](https://github.com/tendermint/tendermint/issues/1148)
    76  * Pubsub-style networking topology and pattern.
    77  * Support multiple chain IDs in the same network.
    78  
    79  ## Status
    80  
    81  Proposed
    82  
    83  ## Consequences
    84  
    85  ### Positive
    86  
    87  * Cleaner, simpler architecture that's easier to reason about and test, and thus hopefully less buggy.
    88  
    89  * Improved performance and robustness.
    90  
    91  * Reduced maintenance burden and increased interoperability by the possible adoption of standardized protocols such as QUIC and Noise.
    92  
    93  * Improved usability, with better observability, simpler configuration, and more automation (e.g. peer/service/address discovery, rate-limiting, and backpressure).
    94  
    95  ### Negative
    96  
    97  * Maintaining our own P2P networking stack is resource-intensive.
    98  
    99  * Abstracting away the underlying transport may prevent usage of advanced transport features.
   100  
   101  * Breaking changes to APIs and protocols are disruptive to users.
   102  
   103  ## References
   104  
   105  See issue links above.
   106  
   107  - [#2067: P2P Refactor](https://github.com/tendermint/tendermint/issues/2067)
   108  
   109  - [P2P refactor brainstorm document](https://docs.google.com/document/d/1FUTADZyLnwA9z7ndayuhAdAFRKujhh_y73D0ZFdKiOQ/edit?pli=1#)