Skip to content

wip: Move EndpointInfo and friends to iroh-dns#4094

Draft
rklaehn wants to merge 12 commits intoinline-pkarrfrom
move-relay-dns-to-dns
Draft

wip: Move EndpointInfo and friends to iroh-dns#4094
rklaehn wants to merge 12 commits intoinline-pkarrfrom
move-relay-dns-to-dns

Conversation

@rklaehn
Copy link
Copy Markdown
Contributor

@rklaehn rklaehn commented Apr 9, 2026

Description

As suggested by @Frando: move more dns related stuff from iroh-relay to iroh-dns.

Builds on top of #4026

Breaking Changes

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:

rklaehn added 2 commits April 9, 2026 16:02
This allows some more things to be come private and avoids an Ext trait.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

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

Last updated: 2026-04-13T12:30:20Z

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

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: d59c19d

@n0bot n0bot bot added this to iroh Apr 9, 2026
@github-project-automation github-project-automation bot moved this to 🚑 Needs Triage in iroh Apr 9, 2026
rklaehn added 6 commits April 9, 2026 16:55
I think this is worth it. It means no more stupid Ext trait or free functions.
And like it or not, z32 is used by multiple discovery providers that we support.

The cost is just

1. a compile time dependency on data-encoding-macro
2. 256+32 bytes of static data

No allocation, no locks, unlike before.
@rklaehn rklaehn force-pushed the move-relay-dns-to-dns branch from 33bc0db to 8985654 Compare April 10, 2026 08:17
@rklaehn rklaehn self-assigned this Apr 10, 2026
@rklaehn rklaehn mentioned this pull request Apr 10, 2026
11 tasks
@rklaehn rklaehn force-pushed the move-relay-dns-to-dns branch from 6d36656 to 6e226d4 Compare April 10, 2026 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚑 Needs Triage

Development

Successfully merging this pull request may close these issues.

1 participant