Added a check to work-around a bug in ESP32's libs (connect returns t…

…rue even when connection actually failed - no endpoint)
gilmaimon · Aug 1, 2019 · ca5ab95 · ca5ab95
1 parent aad6457
commit ca5ab95
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/websockets_client.cpp b/src/websockets_client.cpp
@@ -248,6 +248,12 @@ namespace websockets {
         auto handshake = generateHandshake(internals::fromInterfaceString(host), internals::fromInterfaceString(path), _customHeaders);
         this->_client->send(handshake.requestStr);
 
+        // This check is needed because of an ESP32 lib bug that wont signal that the connection had 
+        // failed in `->connect` (called above), sometimes the disconnect will only be noticed here (after a `send`)
+        if(!available()) {
+            return false;
+        }
+
         auto head = this->_client->readLine();
         if(!doestStartsWith(head, "HTTP/1.1 101")) {
             close(CloseReason_ProtocolError);