Skip to content
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

support configurable interface name for wpantund. #1

Conversation

LeZhang2016
Copy link

No description provided.

@LeZhang2016 LeZhang2016 force-pushed the feature-configurable-interfaceName branch from 9ebb875 to fc87688 Compare May 30, 2017 12:36
@LeZhang2016 LeZhang2016 force-pushed the feature-configurable-interfaceName branch from fc87688 to c6368a2 Compare May 30, 2017 12:42
DuaneEllis-TI added a commit to DuaneEllis-TI/borderrouter that referenced this pull request Jul 28, 2017
…P-158-border-router-git-ignore-missing-non-existent to master

* commit '80ac484a1cb85ecfee9012dfa2f44af6da1077af':
  TIOP-158 | Add gitignore files
jwhui pushed a commit that referenced this pull request Nov 22, 2021
Avahi may rename the meshcop service and publish it when there's a
name conflict. However, our current logic has a bug that it may
publish redundant meshcop services after some renaming.

For example there are two BRs on the infra link:
- BR1 starts. It publishes instance._meshcop._udp.
- BR2 starts. It publishes instance #1._meshcop._udp.
- BR1 stops. It unpublishes instance._meshcop._udp.
- BR2 wants to update the existing meshcop service. However, it
  actually publishes a new service instance._meshcop._udp. There are
  two meshcop services existing at the same time both published by
  BR2.

To fix this bug, we will keep track of the original name in the avahi
publisher. When we call PublishService, we should always pass in the
original name and let the publisher handle the renaming. In this way
we will be able to find the wanted service when updating it or
unpublishing it.
Irving-cl pushed a commit to Irving-cl/ot-br-posix that referenced this pull request Dec 7, 2021
…nthread#1104)

Avahi may rename the meshcop service and publish it when there's a
name conflict. However, our current logic has a bug that it may
publish redundant meshcop services after some renaming.

For example there are two BRs on the infra link:
- BR1 starts. It publishes instance._meshcop._udp.
- BR2 starts. It publishes instance openthread#1._meshcop._udp.
- BR1 stops. It unpublishes instance._meshcop._udp.
- BR2 wants to update the existing meshcop service. However, it
  actually publishes a new service instance._meshcop._udp. There are
  two meshcop services existing at the same time both published by
  BR2.

To fix this bug, we will keep track of the original name in the avahi
publisher. When we call PublishService, we should always pass in the
original name and let the publisher handle the renaming. In this way
we will be able to find the wanted service when updating it or
unpublishing it.

GitOrigin-RevId: d14c225
(cherry picked from commit 33eb9b87edade197db47f857836ef1642d74ee33)
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.

None yet

2 participants