From b0efc0057236874eb252341c377074f0e57fc64b Mon Sep 17 00:00:00 2001
From: Ran Bao <worksev@gmail.com>
Date: Thu, 17 Oct 2024 20:40:40 +1300
Subject: [PATCH] Add no password

---
 src/html/wizard.html | 28 ++++------------------------
 src/wireless.c       | 12 +++++++++---
 2 files changed, 13 insertions(+), 27 deletions(-)

diff --git a/src/html/wizard.html b/src/html/wizard.html
index 8fab882..0039e8d 100644
--- a/src/html/wizard.html
+++ b/src/html/wizard.html
@@ -168,21 +168,6 @@ <h1>Configure WiFi</h1>
 
 
     <script>
-        function validateForm(form) {
-            var isValid = true;
-            allInputs = form.getElementsByTagName("input");
-            for (const input of allInputs) {
-                if (input.value == "") {
-                    isValid = false;
-                    input.classList.add("invalid");
-                }
-                else {
-                    input.classList.remove("invalid");
-                }
-            }
-
-            return isValid;
-        }
 
         // Submit the form data to the action URI
         function putForm(targetForm, saveToEeprom) {
@@ -232,16 +217,11 @@ <h1>Configure WiFi</h1>
 
             const form = document.getElementById("wifi-config")
 
-            // First validate the input
-            const isValid = validateForm(form);
-
-            if (isValid) {
-                putForm(form, true);
+            putForm(form, true);
 
-                // Send reboot request
-                const uri = "/rest/system_control?s5=true";
-                fetch(uri);
-            }
+            // Send reboot request
+            const uri = "/rest/system_control?s5=true";
+            fetch(uri);
         })
 
         function populateWifiConfig() {
diff --git a/src/wireless.c b/src/wireless.c
index 2bd39d7..d34a856 100644
--- a/src/wireless.c
+++ b/src/wireless.c
@@ -291,14 +291,20 @@ void wireless_task(void *p) {
         // Show the current joining SSID
         sprintf(first_line_buffer, ">%s", wireless_config.eeprom_wireless_metadata.ssid);
 
+        // If the authentication method is open then the password is NULL
+        const char * wifi_password = NULL;
+        if (wireless_config.eeprom_wireless_metadata.auth != AUTH_OPEN) {
+            wifi_password = wireless_config.eeprom_wireless_metadata.pw;
+        }
+
         // Retry within timeframe
         TickType_t stop_tick = xTaskGetTickCount() + pdMS_TO_TICKS(wireless_config.eeprom_wireless_metadata.timeout_ms);
         while (xTaskGetTickCount() < stop_tick) {
             int resp;
             resp = cyw43_arch_wifi_connect_timeout_ms(wireless_config.eeprom_wireless_metadata.ssid,
-                                                    wireless_config.eeprom_wireless_metadata.pw,
-                                                    get_cyw43_auth(wireless_config.eeprom_wireless_metadata.auth),
-                                                    wireless_config.eeprom_wireless_metadata.timeout_ms);
+                                                      wifi_password,
+                                                      get_cyw43_auth(wireless_config.eeprom_wireless_metadata.auth),
+                                                      wireless_config.eeprom_wireless_metadata.timeout_ms);
             if (resp == PICO_OK) {
                 wireless_config.current_wireless_state = WIRELESS_STATE_STA_MODE_LISTEN;
                 break;