-
Notifications
You must be signed in to change notification settings - Fork 145
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 not working (Empty Array) #227
Comments
I had a few issues on my network when trying to write this as I have multiple VLANs and pretty complex firewall in place. When debugging my issues I used my firewall to capture the traffic and analyse it to see where things were not traversing as expected/intended. This of course is not particularly simple stuff to work through when it does not work. There are some utilities and tools that you can use also on PC, Linux and MacOS that reveal what mDNS traffic and responses are present on your network like Is your client and bridge on the same network? If there might be some segregation, turning on things like Avahi (or mDNS discover option) on the router can help. |
Yes client and bridge are on the same network. Router is pretty old and has no such options ;). I will try again with firewalls disabled. Otherwise its probably my old routers fault. |
Seeing the same issue as @steffenu using beta 15 and node 19.6 Bridge, router, machine (Mac OS Ventura 13.3 (22E252)) all on the same network. No subnets, VLANs whatsoever... node discoverBridgeViaMDNS.js
[] node discoverBridgeViaNupnp.js
[
{
"ipaddress": "XXX.XXX.X.X",
"config": {
"name": "Philips hue 2",
"ipaddress": "XXX.XXX.X.X",
"modelid": "BSB002",
"swversion": "1957113050"
}
}
] The Any hint or recommendation? |
I have swapped out the That said, it seems to be a little more robust that the pervious version and if you only have one bridge, rather than wait for the time to expire, you can return on discovery but passing in a timeout (in ms) and the boolean I released this change as |
Hi Peter, Unfortunately I can't just use beta 16 'as is', because I've got the problem, where |
@peter-murray I don't know if this is of any help or value, but yesterday I tried to find the bridge using: https://github.com/agnat/node_mdns only using default settings - and it seems to Although I have to admit, I never worked on this before and likely don't know about potential implications (and there might be good reasons other implementations do some more - time-consuming - filtering/searching). I just wanted to mention that finding the brige using the above mentioned library was quick and reliable... |
I had swapped out the library on this branch for |
That specific library, https://github.com/agnat/node_mdns unfortunately has platform specific bindings, so not something I can adopt with the library. |
Do you think this will make it into the main branch any time soon? |
It is on my backlog. To support it I have to port all the tests over to typescript which is painful and slow, as well as needing to address the bridge test lookups to mdns. I am busy with work currently that will not afford me time to get to this until later in the month at best. |
Thanks for the reply Peter - looking forward to it |
This is not giving me any results ( Empty Array )
what can I do ?
the api seems to work fine when I just use my ip directly:
const unauthenticatedApi = await hueApi.createLocal("192.168.1.105").connect();
Ok const results = await discovery.nupnpSearch();
seems to work .. but mdns still doesnt .. which I absolutely would need for realiable discovery :D
I have a pretty old router tplink TL-WR1043ND .. could this cause any issues ?
The text was updated successfully, but these errors were encountered: