Skip to content

Always use multicasts for service discovery#1127

Merged
marcelveldt merged 1 commit intomainfrom
always-use-multicast-browsing
Apr 10, 2025
Merged

Always use multicasts for service discovery#1127
marcelveldt merged 1 commit intomainfrom
always-use-multicast-browsing

Conversation

@agners
Copy link
Collaborator

@agners agners commented Apr 9, 2025

The unicast service discovery leads to unicast responses. However, only a single process can receive a unicast response. This is a problem since the Matter Server is most likely not the only mDNS client on a given host.

By default, the service discovery starts with an unicast request first and continues with multicast requests from then onwards (the first multicast request is sent about 200ms later). So service discovery worked with the default initialization too. But with this change we get rid of the unicast request speeding up the service discovery slightly.

The unicast service discovery leads to unicast responses. However, only
a single process can receive a unicast response. This is a problem
since the Matter Server is most likely not the only mDNS client on a
given host.

By default, the service discovery starts with an unicast request first
and continues with multicast requests from then onwards (the first
multicast request is sent about 200ms later). So service discovery
worked with the default initialization too. But with this change we
get rid of the unicast request speeding up the service discovery
slightly.
@marcelveldt marcelveldt merged commit 679dcd6 into main Apr 10, 2025
3 checks passed
@marcelveldt marcelveldt deleted the always-use-multicast-browsing branch April 10, 2025 09:28
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.

2 participants