Skip to content

basichost: refactor to use DI for some services#3407

Open
sukunrt wants to merge 5 commits intomasterfrom
sukun/basichost-refactor
Open

basichost: refactor to use DI for some services#3407
sukunrt wants to merge 5 commits intomasterfrom
sukun/basichost-refactor

Conversation

@sukunrt
Copy link
Copy Markdown
Member

@sukunrt sukunrt commented Oct 7, 2025

  1. Removes reference to Ping Service, and PeerstoreManager. They're started via the constructor.
  2. Removes relay service manager. The relay service now deregisters the stream handler on reachability change.
  3. Moves natmanager out of basichost. This now works similarly to ObservedAddrsManager.

This reduces the complexity in basichost and lets users opt into only the modules they're actually using.

@sukunrt sukunrt requested a review from MarcoPolo October 7, 2025 15:01
@MarcoPolo MarcoPolo mentioned this pull request Oct 7, 2025
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant