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

Windows DNS Server Order by Route Metrics #81

Merged
merged 2 commits into from Mar 13, 2017
Merged

Windows DNS Server Order by Route Metrics #81

merged 2 commits into from Mar 13, 2017

Commits on Jan 13, 2017

  1. Windows DNS server sorting

    Original Patch From Brad Spencer:
    https://c-ares.haxx.se/mail/c-ares-archive-2016-04/0000.shtml
    
    My modifications include:
     * Dynamically find GetBestRoute2 since it is a Windows Vista+ symbol, and will fall back to prior behavior when not available.
     * Prefer get_DNS_AdaptersAddresses as the modifications should alleviate the concerns which caused us to prefer get_DNS_NetworkParams
     * Update AppVeyor to use MinGW-w64 instead of the legacy MinGW
     * Fix compile error in test suite for Windows.
    
    Original message from patch below:
    
    From: Brad Spencer <bspencer@blackberry.com>
    Date: Fri, 29 Apr 2016 14:26:23 -0300
    
    On Windows, the c-ares DNS resolver tries first to get a full list of
    DNS server addresses by enumerating the system's IPv4/v6 interfaces and
    then getting the per-interface DNS server lists from those interfaces
    and joining them together. The OS, at least in the way the c-ares
    prefers to query them (which also may be the only or best way in some
    environments), does not provide a unified list of DNS servers ordered
    according to "current network conditions". Currently, c-ares will then
    try to use them in whatever order the nested enumeration produces, which
    may result in DNS requests being sent to servers on one interface
    (hosting the current default route, for example) that are only intended
    to be used via another interface (intended to be used when the first
    interface is not available, for example). This, in turn, can lead to
    spurious failures and timeouts simply because of the server address
    order that resulted because of the enumeration process.
    
    This patch makes the (safe?) assumption that there is no other better
    rule to chose which interface's DNS server list should be prioritized.
    After all, a DNS lookup isn't something "per network"; applications
    don't look up "these DNS names on this interface and those DNS names on
    that interface". There is a single resource pool of DNS servers and the
    application should presume that any server will give it the "right"
    answer. However, even if all DNS servers are assumed to give equally
    useful responses, it is reasonable to expect that some DNS servers will
    not accept requests on all interfaces. This patch avoids the problem by
    sorting the DNS server addresses using the Windows IPv4/v6 routing tables.
    
    For example, a request to DNS server C on interface 2 that is actually
    sent over interface 1 (which may happen to have the default route) may
    be rejected by or not delivered to DNS server C. So, better to use DNS
    servers A and B associated with interface 1, at least as a first try.
    
    By using the metric of the route to the DNS server itself as a proxy for
    priority of the DNS server in the list, this patch is able to adapt
    dynamically to changes in the interface list, the DNS server lists per
    interface, which interfaces are active, the routing table, and so on,
    while always picking a good "best" DNS server first.
    
    In cases where any DNS server on any interface will do, this patch still
    seems useful because it will prioritize a lower-metric route's (and thus
    interface's) servers.
    Brad House committed Jan 13, 2017
    Copy the full SHA
    8f252b6 View commit details
    Browse the repository at this point in the history

Commits on Feb 10, 2017

  1. Merge remote-tracking branch 'upstream/master'

    Brad House committed Feb 10, 2017
    Copy the full SHA
    c0bad21 View commit details
    Browse the repository at this point in the history