Release v3.2.4

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 3.2.4
+
+- Correct profile-backed Wi-Fi hand-off from the embedded web portal: the
+  station controller now owns attempts started through either portal surface,
+  preventing the legacy empty-SSID path from reporting success before a
+  station address is available.
+
 ## 3.2.3
 
 - Clarify the public integration boundary, built-in browser protocol, and

README.md

@@ -91,7 +91,7 @@ The [Branded Portal](examples/BrandedPortal/) example includes a static SVG, acc
 ```ini
 [common]
 lib_deps =
-    WiFiManager=https://github.com/alexhopeoconnor/WiFiManager.git#v3.2.3
+    WiFiManager=https://github.com/alexhopeoconnor/WiFiManager.git#v3.2.4
 ```
 
 The suffix after `#` is a Git ref. PlatformIO clones the repository and checks out that release tag; GitHub Release assets are unrelated. Arduino IDE users can install this repository as a library checkout.

docs/GETTING_STARTED.md

@@ -26,7 +26,7 @@ void loop() {
 
 ```ini
 lib_deps =
-    WiFiManager=https://github.com/alexhopeoconnor/WiFiManager.git#v3.2.3
+    WiFiManager=https://github.com/alexhopeoconnor/WiFiManager.git#v3.2.4
 ```
 
 The package includes the asynchronous web and TCP dependencies required by the selected ESP8266 or ESP32 target. Add WiFiManager as the application’s direct dependency; do not copy its internal dependency list into your project.

lib/WiFiManager/include/WiFiManager.h

@@ -1082,6 +1082,12 @@ protected:
         _configPortalStart = millis();
       }
     }
+    void          wmTestSetWebPortalActive(bool active) {
+      webPortalActive = active;
+      if (active) {
+        _configPortalStart = millis();
+      }
+    }
     void          wmTestSetPortalConnectSuccess(const String& message, const String& stationIp, uint8_t status = WL_CONNECTED) {
       _cpConnectState = wm_cp_connect_state_t::success;
       _cpConnectMessage = message;

lib/WiFiManager/src/WiFiManager.cpp

@@ -385,7 +385,10 @@ bool WiFiManager::startStationCandidate(const WiFiManagerStationProfiles& candid
   _begin();
   _stationCandidate = candidate;
   _stationCandidateActive = true;
-  _stationCandidateFromPortal = configPortalActive;
+  // A profile submitted from either portal surface is owned by the station
+  // controller.  Do not let the legacy empty-SSID portal state machine mark
+  // it successful before the controller has a usable station address.
+  _stationCandidateFromPortal = configPortalActive || webPortalActive;
   _stationAttemptMask = 0;
   _stationPendingSlot = WM_NO_STATION_PROFILE;
   _stationNextAttemptAt = 0;
@@ -506,7 +509,7 @@ void WiFiManager::queueStationProfile(uint8_t slot) {
 }
 
 void WiFiManager::completePortalStationAttempt(bool success, uint8_t status, const String& message) {
-  if (!_stationCandidateFromPortal && !configPortalActive) {
+  if (!_stationCandidateFromPortal && !configPortalActive && !webPortalActive) {
     return;
   }
 

library.json

@@ -1,6 +1,6 @@
 {
   "name": "WiFiManager",
-  "version": "3.2.3",
+  "version": "3.2.4",
   "keywords": [
     "wifi",
     "wi-fi",

test/test_wifimanager/test_main.cpp

@@ -14,6 +14,7 @@ TestCase tests[] = {
     TEST_ENTRY(test_api_wifi_meta_password_field_type),
     TEST_ENTRY(test_api_wifi_connect_status_success_redirect),
     TEST_ENTRY(test_profile_portal_success_keeps_handoff_alive),
+    TEST_ENTRY(test_profile_portal_candidate_does_not_take_legacy_empty_ssid_path),
     TEST_ENTRY(test_api_info_json_shape),
     TEST_ENTRY(test_api_params_json_shape),
     TEST_ENTRY(test_api_status_json_shape),
@@ -183,4 +184,3 @@ void loop() {
         begun = false; // avoid repeating
     }
 }
-

test/test_wifimanager/test_main.h

@@ -119,6 +119,7 @@ void test_api_wifi_meta_json_shape();
 void test_api_wifi_meta_password_field_type();
 void test_api_wifi_connect_status_success_redirect();
 void test_profile_portal_success_keeps_handoff_alive();
+void test_profile_portal_candidate_does_not_take_legacy_empty_ssid_path();
 void test_api_info_json_shape();
 void test_api_params_json_shape();
 void test_api_status_json_shape();
@@ -146,4 +147,3 @@ void test_portal_resource_recovery();
 void test_scan_result_storage_released_when_portal_closes();
 
 #endif // TEST_MAIN_H
-

test/test_wifimanager/tests/test_api_json_and_captive.cpp

@@ -96,6 +96,29 @@ void test_profile_portal_success_keeps_handoff_alive() {
     Serial.println("[TEST]   Profile portal success hand-off delay test completed successfully");
 }
 
+void test_profile_portal_candidate_does_not_take_legacy_empty_ssid_path() {
+    Serial.println("[TEST]   Testing profile portal candidate state ownership...");
+
+    WiFiManager wm;
+#ifdef UNIT_TEST
+    WiFiManagerStationProfiles candidate;
+    candidate.slots[0].enabled = true;
+    snprintf(candidate.slots[0].ssid, sizeof(candidate.slots[0].ssid), "%s", "candidate-network");
+    wm.wmTestSetWebPortalActive(true);
+    TEST_ASSERT_TRUE(wm.startStationCandidate(candidate));
+    wm.process();
+    TEST_ASSERT_EQUAL_MESSAGE(
+        WiFiManager::WM_CP_CONNECT_WAITING,
+        wm.getConfigPortalConnectState(),
+        "A web-portal profile candidate must not be treated as the legacy empty-SSID save path"
+    );
+#else
+    TEST_IGNORE_MESSAGE("UNIT_TEST helpers unavailable");
+#endif
+
+    Serial.println("[TEST]   Profile portal candidate state ownership test completed successfully");
+}
+
 void test_api_info_json_shape() {
     Serial.println("[TEST]   Testing /api/info JSON shape...");