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

No responses on Windows #41

Open
panta opened this issue Jun 14, 2017 · 6 comments
Open

No responses on Windows #41

panta opened this issue Jun 14, 2017 · 6 comments

Comments

@panta
Copy link

panta commented Jun 14, 2017

Hi,

I'm using multicast-dns along with bonjour in an Electron app (built with electron-builder).
While on OS X it works like a charm, on Windows machines I see my initial query, but I don't receive any response (the firewall is disabled). On some of these machines, if I launch the Bonjour Browser utility before my app, then the app will subsequently display responses.

Please note that the Windows machines I've tested are VirtualBox VMs and have multiple interfaces (the first NAT, the second bridged - the latter being the one that should receive the responses). Using wireshark I can see the incoming mdns traffic on port 5353 with destination 224.0.0.251.

Is there something I'm patently overlooking, or something I could try?

Thanks in advance.

@panta
Copy link
Author

panta commented Jun 14, 2017

Answering my own question. Seems due to the fact that multicast-dns binds to the first interface it finds. When multiple interfaces are present, it's necessary to listen on all of them (but this requires the multi-interace support of PR #38).

@mafintosh
Copy link
Owner

@panta yea we should get that fixed asap. just need to fix that merge conflict

@panta
Copy link
Author

panta commented Jun 15, 2017

I've fixed it myself if you want, in panta/multicast-dns.

@panta
Copy link
Author

panta commented Jun 15, 2017

Created new PR #42, with the conflict resolved. But the glory should go to @bnielsen1965 for the actual work :)

@mafintosh
Copy link
Owner

Great. Will take a look tonight.

@bnielsen1965
Copy link
Contributor

Mathias,
You have your hands full, so I went ahead and did a major rework of multicast-dns and added it as a new npm module. Assuming I did everything correct it will be mostly backward compatible with multicast-dns and it will handle multiple network interfaces.

I thought you might want to take a look...
https://www.npmjs.com/package/mdns-server

Bryan

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