Skip to content

Conversation

ckhardin
Copy link
Contributor

@ckhardin ckhardin commented Oct 8, 2025

The address resoution from RFC6763 is generally a PTR, SRV, TXT, and an A or AAAA. Records, so this change is mainly to start drawing a clear functional change between the DNS query shell and the DNS service shell that should be more of a "avahi-browse" or "dns-sd" functional style.

So, this is not a very robust implementation since it is a chain of queries when the goal should be to get additional records from the message itself but those changes can be added iteratively.

Basic point of this change is to show the procedure for the "browse"

  • PTR service resolution
  • SRV query from the result of the PTR
  • AAAA (or A) from the result of the SRV

TXT records are ignored for now but should be added into this as additional record support is extended.

@ckhardin ckhardin force-pushed the ckhardin/dns-service-browse-concept-in-the-shell branch 3 times, most recently from 7e00001 to d58eea2 Compare October 8, 2025 23:52
Copy link
Contributor

@rlubos rlubos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally no objections

@ckhardin ckhardin force-pushed the ckhardin/dns-service-browse-concept-in-the-shell branch from d58eea2 to 08b2286 Compare October 9, 2025 18:13
The address resoution from RFC6763 is generally a PTR, SRV, TXT, and
an A or AAAA. Records, so this change is mainly to start drawing a
clear functional change between the DNS query shell and the DNS
service shell that should be more of a "avahi-browse" or "dns-sd"
functional style.

So, this is not a very robust implementation since it is a chain of
queries when the goal should be to get additional records from the
message itself but those changes can be added iteratively.

Basic point of this change is to show the procedure for the "browse"

- PTR service resolution
- SRV query from the result of the PTR
- AAAA (or A) from the result of the SRV

TXT records are ignored for now but should be added into this as
additional record support is extended.

Signed-off-by: Charles Hardin <[email protected]>
@ckhardin ckhardin force-pushed the ckhardin/dns-service-browse-concept-in-the-shell branch from 08b2286 to a3614c1 Compare October 13, 2025 22:02
Copy link

@jhedberg jhedberg merged commit 7158f33 into zephyrproject-rtos:main Oct 15, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants