Add restart post-AP to prevent issues with launching HTTP server

thorrak · Feb 12, 2021 · 9956410 · 9956410
1 parent 9208067
commit 9956410
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/bin/d32_pro_tft_firmware.bin b/bin/d32_pro_tft_firmware.bin
diff --git a/bin/lcd_ssd1306_firmware.bin b/bin/lcd_ssd1306_firmware.bin
diff --git a/bin/tft_espi_firmware.bin b/bin/tft_espi_firmware.bin
diff --git a/src/wifi_setup.cpp b/src/wifi_setup.cpp
@@ -83,14 +83,15 @@ void initWiFi() {
             if (url.isValidHostName(custom_mdns_name.getValue())) {
                 strlcpy(config.mdnsID, custom_mdns_name.getValue(), 31);
                 saveConfig();
-                // Doing this to reset the DHCP name, the portal should never pop
-                ESP.restart();
             } else {
-                // If the mDNS name is invalid, reset the WiFi configuration and restart
-                // the ESP8266
+                // If the mDNS name is invalid, reset the WiFi configuration and restart the ESP8266
                 disconnectWiFi();
             }
         }
+        // Doing this to reset the DHCP name, the portal should never pop
+        // Additionally, there is a bug where the HTTP server doesn't spin up after the AP shuts down. Not sure where
+        // that issue is, but this solves it.
+        ESP.restart();
     }
 
     if (!MDNS.begin(config.mdnsID)) {