From 5f1a2631583689d237c0718093a910eb39831777 Mon Sep 17 00:00:00 2001
From: Harald Barth <haba@kth.se>
Date: Fri, 5 Aug 2022 19:54:01 +0200
Subject: [PATCH] do not make delete client conditional

---
 WifiESP32.cpp | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/WifiESP32.cpp b/WifiESP32.cpp
index 770557e..3be3827 100644
--- a/WifiESP32.cpp
+++ b/WifiESP32.cpp
@@ -193,15 +193,17 @@ void WifiESP::loop() {
 
   // really no good way to check for LISTEN especially in AP mode?
   if (APmode || WiFi.status() == WL_CONNECTED) {
-    if (server->hasClient()) {
-      // loop over all clients and remove inactive
-      for (clientId=0; clientId<clients.size(); clientId++){
-	// check if client is there and alive
-	if(!clients[clientId].connected()) {
-	  clients[clientId].stop();
-	  clients.erase(clients.begin()+clientId);
-	}
+    // loop over all clients and remove inactive
+    for (clientId=0; clientId<clients.size(); clientId++){
+      // check if client is there and alive
+      if(!clients[clientId].connected()) {
+	DIAG(F("Remove client %d %s"), clientId, clients[clientId].remoteIP().toString().c_str());
+	CommandDistributor::forget(clientId);
+	clients[clientId].stop();
+	clients.erase(clients.begin()+clientId);
       }
+    }
+    if (server->hasClient()) {
       WiFiClient client;
       while (client = server->available()) {
 	clients.push_back(client);