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.

Sign up for GitHub

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

Jump to bottom

Mdns multi interface #2864

Merged
merged 2 commits into from
Jan 17, 2017
Merged

Mdns multi interface #2864

merged 2 commits into from
Jan 17, 2017

Commits on Jan 15, 2017

  1. Revert "ESP8266mDNS using the provided IP in the begin method (esp826… 

    …6#2349)"

Manually specifying the AP IP isn't required; the next change will modify
the MDNS code to correctly handle any combination of AP and STA modes, and
correctly respond to requests on all active interfaces.

This reverts commit b682d59.
    @swarren
    swarren committed Jan 15, 2017
    Configuration menu
    Copy the full SHA
    ad0691d View commit details
    Browse the repository at this point in the history

  2. ESP8266mDNS: support AP and STA interfaces at once 

    Bind the UDP connection to IP_ADDR_ANY rather than a specific interface
IP, so that it can receive queries from all interfaces at once.

When processing a query, detect which interface it was received on, and
pass this information to the reply code. Set the IP multicast interface
before each transmission, so that we can route each packet to a
different interface if required. These changes enable the code to
respond correctly on both AP and STA interfaces at once. The original
code only worked correctly for the STA interface if both were enabled.

When advertizing all services, do it on both AP and STA interfaces if
they're both enabled.

Provide an API for the application to notify the MDNS code if the AP
configuration changes (enabled, disabled, IP changed). Ideally, the WiFi
core would provide an event callback for this purpose, as it does for
STA changes, but it does not appear to, so the application must provide
this information.
    @swarren
    swarren committed Jan 15, 2017
    Configuration menu
    Copy the full SHA
    3e19c6e View commit details
    Browse the repository at this point in the history