Skip to content

Conversation

Frando
Copy link
Member

@Frando Frando commented Oct 13, 2025

Description

  • Removes Endpoint::add_node_addr_with_source
  • Changes visibility of Endpoint::add_node_addr from private to pub(crate) and add a source argument, and uses it in discovery instead of the now-removed add_node_addr_with_source
  • Refactors a few tests to not use Endpoint::add_node_addr anymore

The only remaining uses of Endpoint::add_node_addr are now in Endpoint::connect (when passing a NodeAddr) and in the task processing discovery results. Yay!

Breaking Changes

  • iroh::Endpoint::add_node_addr_with_source is removed. Use a discovery service instead.

Notes & open questions

Change checklist

  • Self-review.
  • Documentation updates following the style guide, if relevant.
  • Tests if relevant.
  • All breaking changes documented.
    • List all breaking changes in the above "Breaking Changes" section.
    • Open an issue or PR on any number0 repos that are affected by this breaking change. Give guidance on how the updates should be handled or do the actual updates themselves. The major ones are:

Copy link

github-actions bot commented Oct 13, 2025

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/3528/docs/iroh/

Last updated: 2025-10-14T10:22:20Z

Copy link

github-actions bot commented Oct 13, 2025

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: 3117bde

@n0bot n0bot bot added this to iroh Oct 13, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Oct 13, 2025
@dignifiedquire dignifiedquire changed the title refactor: finish removal of add_node_addr from public api refactor(iroh)!: finish removal of add_node_addr from public api Oct 13, 2025
@dignifiedquire dignifiedquire added this pull request to the merge queue Oct 14, 2025
Merged via the queue into main with commit c2fce51 Oct 14, 2025
51 of 54 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants