From 92a17606130dfdc0de5f5236670ee79f3b1647d4 Mon Sep 17 00:00:00 2001
From: Juraj Andrassy <juraj.andrassy@gmail.com>
Date: Thu, 9 Nov 2023 16:56:38 +0100
Subject: [PATCH] WiFiServer - deprecate available(), use accept() everywhere

esp8266 and esp32 server.available() is not implemented as documented
it is implemented like server.accept() is documented by Arduino
---
 .../ESPmDNS/examples/mDNS_Web_Server/mDNS_Web_Server.ino      | 2 +-
 libraries/WebServer/src/WebServer.cpp                         | 2 +-
 libraries/WiFi/examples/SimpleWiFiServer/SimpleWiFiServer.ino | 2 +-
 libraries/WiFi/examples/WiFiAccessPoint/WiFiAccessPoint.ino   | 2 +-
 .../WiFi/examples/WiFiTelnetToSerial/WiFiTelnetToSerial.ino   | 4 ++--
 libraries/WiFi/src/WiFiServer.cpp                             | 4 ++++
 libraries/WiFi/src/WiFiServer.h                               | 4 ++--
 7 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/libraries/ESPmDNS/examples/mDNS_Web_Server/mDNS_Web_Server.ino b/libraries/ESPmDNS/examples/mDNS_Web_Server/mDNS_Web_Server.ino
index 946f51aacc1..0da791adc3e 100644
--- a/libraries/ESPmDNS/examples/mDNS_Web_Server/mDNS_Web_Server.ino
+++ b/libraries/ESPmDNS/examples/mDNS_Web_Server/mDNS_Web_Server.ino
@@ -69,7 +69,7 @@ void setup(void)
 void loop(void)
 {
     // Check if a client has connected
-    WiFiClient client = server.available();
+    WiFiClient client = server.accept();
     if (!client) {
         return;
     }
diff --git a/libraries/WebServer/src/WebServer.cpp b/libraries/WebServer/src/WebServer.cpp
index ed0fcf16ed9..d57ec51f6ee 100644
--- a/libraries/WebServer/src/WebServer.cpp
+++ b/libraries/WebServer/src/WebServer.cpp
@@ -274,7 +274,7 @@ void WebServer::serveStatic(const char* uri, FS& fs, const char* path, const cha
 
 void WebServer::handleClient() {
   if (_currentStatus == HC_NONE) {
-    _currentClient = _server.available();
+    _currentClient = _server.accept();
     if (!_currentClient) {
       if (_nullDelay) {
         delay(1);
diff --git a/libraries/WiFi/examples/SimpleWiFiServer/SimpleWiFiServer.ino b/libraries/WiFi/examples/SimpleWiFiServer/SimpleWiFiServer.ino
index 4b9b91e82d8..1867533662c 100644
--- a/libraries/WiFi/examples/SimpleWiFiServer/SimpleWiFiServer.ino
+++ b/libraries/WiFi/examples/SimpleWiFiServer/SimpleWiFiServer.ino
@@ -63,7 +63,7 @@ void setup()
 }
 
 void loop(){
- WiFiClient client = server.available();   // listen for incoming clients
+ WiFiClient client = server.accept();   // listen for incoming clients
 
   if (client) {                             // if you get a client,
     Serial.println("New Client.");           // print a message out the serial port
diff --git a/libraries/WiFi/examples/WiFiAccessPoint/WiFiAccessPoint.ino b/libraries/WiFi/examples/WiFiAccessPoint/WiFiAccessPoint.ino
index d49ea9cbe8c..6c113fe48b9 100644
--- a/libraries/WiFi/examples/WiFiAccessPoint/WiFiAccessPoint.ino
+++ b/libraries/WiFi/examples/WiFiAccessPoint/WiFiAccessPoint.ino
@@ -48,7 +48,7 @@ void setup() {
 }
 
 void loop() {
-  WiFiClient client = server.available();   // listen for incoming clients
+  WiFiClient client = server.accept();   // listen for incoming clients
 
   if (client) {                             // if you get a client,
     Serial.println("New Client.");           // print a message out the serial port
diff --git a/libraries/WiFi/examples/WiFiTelnetToSerial/WiFiTelnetToSerial.ino b/libraries/WiFi/examples/WiFiTelnetToSerial/WiFiTelnetToSerial.ino
index f9b9caf0910..92e41025e98 100644
--- a/libraries/WiFi/examples/WiFiTelnetToSerial/WiFiTelnetToSerial.ino
+++ b/libraries/WiFi/examples/WiFiTelnetToSerial/WiFiTelnetToSerial.ino
@@ -78,7 +78,7 @@ void loop() {
         //find free/disconnected spot
         if (!serverClients[i] || !serverClients[i].connected()){
           if(serverClients[i]) serverClients[i].stop();
-          serverClients[i] = server.available();
+          serverClients[i] = server.accept();
           if (!serverClients[i]) Serial.println("available broken");
           Serial.print("New client: ");
           Serial.print(i); Serial.print(' ');
@@ -88,7 +88,7 @@ void loop() {
       }
       if (i >= MAX_SRV_CLIENTS) {
         //no free/disconnected spot so reject
-        server.available().stop();
+        server.accept().stop();
       }
     }
     //check clients for data
diff --git a/libraries/WiFi/src/WiFiServer.cpp b/libraries/WiFi/src/WiFiServer.cpp
index db21858125b..ef59cf14d6d 100644
--- a/libraries/WiFi/src/WiFiServer.cpp
+++ b/libraries/WiFi/src/WiFiServer.cpp
@@ -39,6 +39,10 @@ size_t WiFiServer::write(const uint8_t *data, size_t len){
 void WiFiServer::stopAll(){}
 
 WiFiClient WiFiServer::available(){
+  return accept();
+}
+
+WiFiClient WiFiServer::accept(){
   if(!_listening)
     return WiFiClient();
   int client_sock;
diff --git a/libraries/WiFi/src/WiFiServer.h b/libraries/WiFi/src/WiFiServer.h
index 346986abad5..b9db726b9b7 100644
--- a/libraries/WiFi/src/WiFiServer.h
+++ b/libraries/WiFi/src/WiFiServer.h
@@ -45,8 +45,8 @@ class WiFiServer : public Server {
       log_v("WiFiServer::WiFiServer(addr=%s, port=%d, ...)", addr.toString().c_str(), port);
     }
     ~WiFiServer(){ end();}
-    WiFiClient available();
-    WiFiClient accept(){return available();}
+    WiFiClient available() __attribute__((deprecated("Renamed to accept().")));
+    WiFiClient accept();
     void begin(uint16_t port=0);
     void begin(uint16_t port, int reuse_enable);
     void setNoDelay(bool nodelay);