-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Android] - Devices with android 12 and below have problem with mDNS discovery of commissionable devices #32686
Comments
Hi, thanks, I can confirm that this fix solve mDns search, and it works way better on all devices (I have pulled all commits from master). Confirmed using chip tool app But it doesn't work for my implementation in our app. After debugging I have succeeded to reproduce crash on chiptool app as well. Root cause is flag setSkipCommissioningComplete(true) in ControllerParams. Shipset of crash is below, full log is attached
|
Close it now. Feel free to reopen if it is still problematic. thanks |
Reproduction steps
General idea is that in our application we want to scan all devices that have commissioning window open and present them to user to choose which one to configure. This works properly with android 13 and 14, but 12 and lower will find just one device.
Array of found devices is filled with same device, second device will be found only when first device close its commissioning mode
When I do exact same steps on android 13 and 14, discovery will add same device 2 or 3 times but it will find second device while both device are in commissioning mode
Issue is tested and mdns discovery is not working properly on
Devices that mdns discovery is working properly
Bellow are logs from chiptool android app from Huawei P smart. As attachment I have added logs from xiaomi tab where everything works properly for comparation purpose.
Command used from ChipDeviceController is public void discoverCommissionableNodes()
huawei p smart.txt
xiaomi.txt
Platform
android
Platform Version(s)
v1.2.0.1
Type
Manually tested with SDK
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: