Skip to content

Changes for non-AsyncDNS and station IP address reporting. #1

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wzaggle
Copy link
Owner

@wzaggle wzaggle commented Aug 30, 2018

Added call to Loop while waiting for connection. loop calls safeLoop that conditionally calls dnsServer->processNextRequest() if not using ESPAsyncDNS.

Moved the WiFi.mode(WIFI_STA) change back to Station Only mode after the successful connect so that the InfoPage can contain the actual IP assigned to the station by the connected AccessPoint. Added additional delays waiting for the infoPage request from the Browser and some additional time on the Browser Refresh to give the station time to actually connect.
This could all probably be done more synchronously with a scripted XMLHttpRequest in place of the current delay and hope method waiting for a connection.

Added call to Loop while waiting for connection.  loop calls safeLoop that conditionally calls dnsServer->processNextRequest() if not using ESPAsyncDNS.

Moved change back to Station Only mode after connect so InfoPage can contain the actual IP assigned to the station by the connected AccessPoint.  Added additional delays waiting for the infoPage request from the Browser and some additional time on the Browser Refresh to give the station time to actually connect.
This could all probably be done more synchronously with a scripted XMLHttpRequest in place of the current delay and hope method waiting for a connection.
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 this pull request may close these issues.

1 participant