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

mDNS Discovery #221

Closed
drewterry opened this issue May 15, 2022 · 4 comments
Closed

mDNS Discovery #221

drewterry opened this issue May 15, 2022 · 4 comments

Comments

@drewterry
Copy link

According to Hue documentation uPNP discovery will be discontinued Q2 2022 (now). Is there a way to do this with node-hue-api?

NOTE 1 – UPnP (or more specifically the UPnP subset Simple Service Discovery Protocol (SSDP)) used to be available as another discovery method but has been deprecated in favor of mDNS and will be disabled in Q2 2022

@peter-murray
Copy link
Owner

The library has supported the SSDP discovery along with the newer remote API discovery endpoint once it became available. I suspect that they will be removing this functionality from the actual bridges, and I have not successfully testing this discovery method for some time now, instead favouring the discovery endpoint myself for a while.

The existing SSDP discovery was exposed under the upnpSearch() function and the newer discovery endpoint based one was nupnpSearch()

I will look into adding support for mDNS discovery and see what is needed in that space.

@peter-murray
Copy link
Owner

Just an update, I have added mDNS locally to the library, but really not getting reliable of fast results inside my network when using it in practice. I will continue to work on this due to the problems with the rate limiting on the discovery endpoint as this is also impacting my test suites.

@peter-murray
Copy link
Owner

I have just released version 5.0.0-beta.14 to npm registry, it includes a new discovery.mdnsSearch(), which at long last appears to be reliable on my network, please check it out and report back any issues or problems.

@pindab0ter
Copy link

I tested it today and it works great! Thanks for the update!

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

No branches or pull requests

3 participants