-
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
can not pass the [TC-DD-2.1] in Android platform #22251
Comments
@nicelyjust Please update the bug with full description of steps and requirements. Specifically what are Please provide a list of commands for me to execute to reproduce the issue. |
The test seems to use RFC 6763 |
How can I run the tv-ap or tv-casting app to verify what is going on? |
Hi Andy @andy31415 : SDK SHA: f046787
|
yes, I think so |
@nicelyjust I have never run tv-app or tv-casting-app ... could you help with instructions on running? |
I was told https://github.com/project-chip/connectedhomeip/blob/master/examples/tv-casting-app/android/README.md exists ... so that seems a good start, will try it. |
Dnssd logic seems to have been added in #9884 by @xylophone21 . It uses NsdManger to register services. I am not sure if the behavior we are seeing is a regression or it never fully advertised. I believe it does advertise at boot, however I do not see NsdManager responding to queries that are done once the application runs. |
It seems that service publishing does not keep a MulticastLock (but resolution does), so publishing is not persistent. After fixing that, publish works however browsing for services will NOT return |
I see:
|
Regarding the test, https://www.rfc-editor.org/rfc/rfc6763.html#section-9 does not seem to say that subtypes are to be enumerated (but does not prevent it either). I believe the test case needs updating or accepting alternate ways of verifying (like resolving the direct subtype like I did above). |
I also encountered the same problem, running the instruction "avahi-browse _services._dns-sd. _udp-r" failed, what is the specific solution? |
the fix improves the Android mDNS behavior but it does not address the issue with TC-DD-2.1 and TC-SC-4.1 since Android still does not respond to subtype queries. The spec does not state that subtype queries are mandatory so probably the solution is to update TC-DD-2.1 and TC-SC-4.1 since these are currently mandatory. |
The fact that NsdManager cannot properly respond to the subtype query in some situations does not mean we need to remove/update the spec. It is possible on Android to be directly responding to particular queries, without going through NsdManager. Furthermore, NsdManager was improved in API level 33 and 34, and I would recommend that we separate platform support from spec requirements. |
Adding my comment here from the linked issue for context:
The service enumeration indicates that the device is meant to respond with its PTR records. Subtype PTRs are PTRs. IMO, the argument that the DUT does not need to respond to queries for the subtypes per the spec is incorrect. These were always meant to be mandatory. In fact the _T subtype is the exception that proves the rule here - it is the only subtype marked as optional. If this is somehow unclear in the spec, we should clarify in the spec. As far as the test failure, one question that really needs to be answered is whether the device actually RESPONDS to the appropriate subtypes even if it fails to list them in the service enumeration. If this is just a service enumeration issue, then perhaps we can adjust the test because the RFC is a LITTLE fudgy there on what's required. If they are not listed in the service enumeration because the device is not responding, then that is a different problem and that's what needs fixing. |
Problem
when tested [TC-DD-2.1]
Announcement by Device Verification [DUT - Commissionee]
through tv-app or tv-casting-app , failed in step #14b &14c , when execute the below command in chip-tool ,can not get the Expected Outcome
Proposed Solution
I'm not sure whether is a problem of the DNS-sd implement in Android platform, Appreciated for any response.
The text was updated successfully, but these errors were encountered: