search

w3c / wot-testing

Repository for WoT Testing purposes
15 stars 29 forks source link

Update DNS-SD UDP discovery results #576

Closed JKRhb closed 1 year ago

JKRhb commented 1 year ago

Following up on @mmccool's email, this PR updates the DNS-SD results for UDP by updating the assertion name that is used in the manual.csv files for node-wot and my DNS-DS CoAP server implementation. This should keep the assertion covered as it was before.

I hope the updates are made to the right files, otherwise, let me know!

mmccool commented 1 year ago

Is this correct? It shows node-wot as ONLY implementing UDP-based DNS-SD now, not TCP. I can merge for now to see how it updates the results (in particular, whether taking out support for TCP-based DNS breaks that assertion) but please double-check.

mmccool commented 1 year ago

Also, right files aren't updated, but don't worry about it, I will fix.

JKRhb commented 1 year ago

Is this correct? It shows node-wot as ONLY implementing UDP-based DNS-SD now, not TCP. I can merge for now to see how it updates the results (in particular, whether taking out support for TCP-based DNS breaks that assertion) but please double-check.

I might be mistaken, but I think there is no support for DNS-SD in the HTTP binding yet, therefore only the UDP-related assertions are satisfied by it (via the CoAP binding). But maybe @relu91, @danielpeintner, and @egekorkan can confirm this?

mmccool commented 1 year ago

OK. Let me merge for now, since what's here is conservative. I think other implementations cover the TCP-based DNS discovery.

relu91 commented 1 year ago

Yes, I can confirm there is no support for TCP-based DNS discovery in node-wot. But there should be support in other implementations (like Zion and another TDD should have support for it).