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-repeater does not set TTL to 255 #10

Closed
JeffFaer opened this issue Dec 1, 2021 · 2 comments · Fixed by #11
Closed

mdns-repeater does not set TTL to 255 #10

JeffFaer opened this issue Dec 1, 2021 · 2 comments · Fixed by #11

Comments

@JeffFaer
Copy link
Contributor

JeffFaer commented Dec 1, 2021

Poking around a bit, this looks like this is the official repository for mdns-repeater. Please correct me if I'm wrong, I saw a couple other copies of it floating around in github


I've been splitting my home network into multiple VLANs for security. I use a Unifi Security Gateway (USG) as a router. USGs offer mdns-repeater to rebroadcast mDNS packets between VLANs. I ran into an issue with mdns-repeater on the USG, and after some digging I think I've finally tracked down the problem: https://community.ui.com/questions/mDNS-repeater-doesnt-work-but-mDNS-reflector-does/646a3583-a510-4296-9b46-84c10bbe94be

The mDNS packets coming from mdns-repeater have an IP TTL of 1, when it's supposed to be 255: https://www.wireshark.org/lists/wireshark-bugs/201002/msg00513.html

All Multicast DNS responses (including responses sent via unicast) SHOULD be sent with IP TTL set to 255.

mdns-repeater should explicitly set an IP TTL of 255

I'm going to take a crack at resolving this, unless there are any objections

@kennylevinsen
Copy link
Collaborator

Poking around a bit, this looks like this is the official repository for mdns-repeater.

I guess what matters is what people are using. This fork of geekman's original code adds whitelist/blacklist and some fixes. If your mdns-repeater has that feature, then it might be this one.

@JeffFaer
Copy link
Contributor Author

JeffFaer commented Dec 3, 2021

Ah, not sure how I missed that as the original one. I'll go ahead and try to patch both 🤷

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

Successfully merging a pull request may close this issue.

2 participants