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

Filter response address records by network interface #40

Open
cyanfish opened this issue Jul 9, 2024 · 2 comments
Open

Filter response address records by network interface #40

cyanfish opened this issue Jul 9, 2024 · 2 comments
Labels
bug Something isn't working good issue A priority to fix

Comments

@cyanfish
Copy link

cyanfish commented Jul 9, 2024

Per https://datatracker.ietf.org/doc/html/rfc6762#section-6.2:

When a Multicast DNS responder sends a Multicast DNS response message
containing its own address records, it MUST include all addresses
that are valid on the interface on which it is sending the message,
and MUST NOT include addresses that are not valid on that interface
(such as addresses that may be configured on the host's other
interfaces).

I have this mostly working in my fork but I'm not sure it's the best way to handle this.

@jdomnitz jdomnitz added bug Something isn't working good issue A priority to fix labels Jul 9, 2024
@jdomnitz
Copy link
Owner

jdomnitz commented Jul 9, 2024

I agree this will need some work but it's worthwhile. I'll work through some other concepts. This is the last area I've found where not complying with the standard causes problems.

@jdomnitz
Copy link
Owner

Reference here: richardschneider#25

Current Issues:

  • IPv6 queries should return IPv4 addresses and vice-versa (They may be routable)
  • Removing all but the sending NIC breaks support for multi-homing and shared services
  • This only works with semetric routing

Easy first steps:

  • Loopback addresses are never routable externally
  • Update service records in all instances where they may change (WIP)

TBD - How does mDNSResponder handle this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good issue A priority to fix
Projects
None yet
Development

No branches or pull requests

2 participants