Split socket handling out of CConnman #30694

1. 
   vasild commented at 9:53 am on August 22, 2024: contributor

   Please describe the feature you’d like to see added.

   Split CConnman in two pieces:

   • protocol agnostic low level socket and connection handler and
   • bitcoin-p2p protocol specific handler.

   This way the code should become more modular and easier to maintain.

   Is your feature related to a problem, if so please describe it.

   Code has intermixed low level socket handling and P2P logic.

   Describe the solution you’d like

   Split CConnman in such a way that the code is easier to read and maintain.

   Describe any alternatives you’ve considered

   Duplicate code.

2. vasild added the label Feature on Aug 22, 2024
3. 
   vasild commented at 9:55 am on August 22, 2024: contributor

   I am looking into doing this.

   cc @Sjors, @pinheadmz

4. 
   pinheadmz commented at 10:17 am on August 22, 2024: member

   I am looking into doing this.

   Wow thanks! My HTTP branch is almost done but is very messy. You can see me copy/paste/edit socket handling in

   https://github.com/pinheadmz/bitcoin/blob/http-netbase/src/http.cpp GenerateEventsPerSock() HandleConnections() AcceptConnections() etc…

5. 
   vasild commented at 7:58 am on September 27, 2024: contributor

   Done in #30988.
6. 
   fanquake commented at 4:54 pm on December 5, 2025: member

   Can you update the issue description to remove mentions of SV2 (given it doesn’t need or uses this), and probably the libevent removal, given that seems to be taking a different approach now? I guess this could also be closed, given all relevant discussion is happening in the PR (#30988).
7. 
   vasild commented at 5:57 am on December 10, 2025: contributor

   Updated the OP and closing as suggested.
8. vasild closed this on Dec 10, 2025

   ---

Contributors
vasild pinheadmz fanquake

Labels
Feature