-
Notifications
You must be signed in to change notification settings - Fork 218
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[mdns] let avahi use RecordBrowser
to resolve addresses of a service
#1875
Conversation
174fbe7
to
967a1f6
Compare
967a1f6
to
7519c76
Compare
RecordBrowser
to resolve addresses of a serviceRecordBrowser
to resolve addresses of a service
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
openthread/openthread#9075 covers the test case. https://github.com/openthread/openthread/actions/runs/5043589092 Shows an example that the previous implementation cannot handle this. |
7519c76
to
df9fa36
Compare
df9fa36
to
fcc8a0a
Compare
fcc8a0a
to
0a3fd93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Avahi only returns one of the host's addresses in the service resolution callback. However, the address may be link-local so it may not be preferred from Thread's perspective. We want to go through the complete list of addresses associated with the host and choose a routable address. Therefore we use
RecordBrowser
which can do this job.