@alteriom/painlessmesh 1.9.2 → 1.9.4

package/CHANGELOG.md

@@ -5,6 +5,31 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.9.4] - 2025-12-03
+
+### Fixed
+
+- **TCP Server Initialization Order** (#219) - Fixed TCP connection error -14 when mesh nodes connect to bridge
+  - **Root Cause**: TCP server was being initialized before the AP interface was configured in `init()`, causing it to bind incorrectly and reject incoming connections from mesh nodes
+  - **Symptom**: Regular mesh nodes could discover the bridge AP and get an IP, but TCP connections failed with error -14 (connection refused)
+  - **Solution**: Reordered initialization in `init()` to start TCP server after AP is configured:
+    - `eventHandleInit()` → `apInit(nodeId)` → `tcpServerInit()` (previously tcpServerInit was first)
+  - Added 100ms stabilization delay in `initAsBridge()` and `initAsSharedGateway()` after mesh init
+  - **Impact**: Fixes `sendToInternet` not working because nodes couldn't establish mesh connections to the bridge
+
+## [1.9.3] - 2025-12-02
+
+### Fixed
+
+- **sendToInternet Example Compilation** - Fixed two compilation errors in the sendToInternet example:
+  - Added missing `addTask(callback, delay)` overload in mesh.hpp for one-shot delayed tasks
+  - Fixed `sendWithPriority` template functions in router.hpp to use `Variant(&package)` instead of `Variant(package)`, enabling proper construction from any PackageInterface-derived class including GatewayDataPackage
+  - **Impact**: The sendToInternet example now compiles correctly with Arduino IDE
+
+### Added
+
+- **sendToInternet Example** - Added to library.json examples list for better discoverability
+
 ## [1.9.2] - 2025-12-01
 
 ### Changed

package/library.json

@@ -6,7 +6,7 @@
     "type": "git",
     "url": "https://github.com/Alteriom/painlessMesh"
   },
-  "version": "1.9.2",
+  "version": "1.9.4",
   "frameworks": [
     "arduino"
   ],
@@ -88,6 +88,7 @@
     "examples/otaReceiver/otaReceiver.ino",
     "examples/otaSender/otaSender.ino",
     "examples/priority/priority_basic_example.ino",
+    "examples/sendToInternet/sendToInternet.ino",
     "examples/sharedGateway/sharedGateway.ino",
     "examples/startHere/startHere.ino",
     "examples/webServer/webServer.ino"

package/library.properties

@@ -1,5 +1,5 @@
 name=Alteriom PainlessMesh
-version=1.9.2
+version=1.9.4
 author=Coopdis,Scotty Franzyshen,Edwin van Leeuwen,Germán Martín,Maximilian Schwarz,Doanh Doanh,Alteriom
 maintainer=Alteriom
 sentence=A painless way to setup a mesh with ESP8266 and ESP32 devices with Alteriom extensions

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alteriom/painlessmesh",
-  "version": "1.9.2",
+  "version": "1.9.4",
   "description": "painlessMesh is a user-friendly library for creating mesh networks with ESP8266 and ESP32 devices. This Alteriom fork includes additional packages for sensor data (SensorPackage), device commands (CommandPackage), and status monitoring (StatusPackage). It handles routing and network management automatically, so you can focus on your application. The library uses JSON-based messaging and syncs time across all nodes, making it ideal for coordinated behaviour like synchronized light displays or sensor networks reporting to a central node.",
   "keywords": [
     "arduino",

package/src/arduino/wifi.hpp

@@ -270,7 +270,6 @@ class Mesh : public painlessmesh::Mesh<Connection> {
       }
     });
 
-    tcpServerInit();
     eventHandleInit();
 
     _apIp = IPAddress(0, 0, 0, 0);
@@ -278,6 +277,12 @@ class Mesh : public painlessmesh::Mesh<Connection> {
     if (connectMode & WIFI_AP) {
       apInit(nodeId);  // setup AP
     }
+    
+    // Initialize TCP server AFTER AP is configured
+    // This ensures the network interfaces are ready when the server starts
+    // Fixes TCP connection error -14 when nodes try to connect to bridge
+    tcpServerInit();
+    
     if (connectMode & WIFI_STA) {
       this->initStation();
     }
@@ -394,6 +399,10 @@ class Mesh : public painlessmesh::Mesh<Connection> {
     init(meshSSID, meshPassword, baseScheduler, port, WIFI_AP_STA, 
          detectedChannel, 0, MAX_CONN);
     
+    // Allow network stack to stabilize after AP and TCP server initialization
+    // This prevents TCP connection errors (-14) when nodes connect
+    delay(100);
+    
     Log(STARTUP, "Step 3: Establishing bridge connection...\n");
     
     // Step 3: Re-establish router connection using stationManual
@@ -565,6 +574,10 @@ class Mesh : public painlessmesh::Mesh<Connection> {
     this->setScheduler(userScheduler);
     init(meshPrefix, meshPassword, port, WIFI_AP_STA, detectedChannel, 0, MAX_CONN);
     
+    // Allow network stack to stabilize after AP and TCP server initialization
+    // This prevents TCP connection errors (-14) when nodes connect
+    delay(100);
+    
     Log(STARTUP, "Step 3: Establishing router connection in shared gateway mode...\n");
     
     // Step 3: Establish router connection using stationManual

package/src/painlessmesh/mesh.hpp

@@ -3038,6 +3038,19 @@ class Mesh : public ntp::MeshTime, public plugin::PackageHandler<T> {
     return plugin::PackageHandler<T>::addTask((*this->mScheduler), aCallback);
   }
 
+  /**
+   * Add a one-shot delayed task
+   *
+   * @param aCallback Function to call after the delay
+   * @param delayMs Delay in milliseconds before executing the callback
+   * @return Shared pointer to the task
+   */
+  inline std::shared_ptr<Task> addTask(std::function<void()> aCallback,
+                                       unsigned long delayMs) {
+    return plugin::PackageHandler<T>::addTask((*this->mScheduler), delayMs,
+                                              TASK_ONCE, aCallback);
+  }
+
   ~Mesh() {
     this->stop();
     if (!isExternalScheduler) delete mScheduler;

package/src/painlessmesh/router.hpp

@@ -67,7 +67,7 @@ bool send(protocol::Variant&& variant, std::shared_ptr<U> conn,
 // New priority-level send functions (0-3 priority levels)
 template <class T, class U>
 bool sendWithPriority(T& package, std::shared_ptr<U> conn, uint8_t priorityLevel) {
-  painlessmesh::protocol::Variant variant(package);
+  painlessmesh::protocol::Variant variant(&package);
   TSTRING msg;
   variant.printTo(msg);
   return conn->addMessageWithPriority(msg, priorityLevel);
@@ -75,7 +75,7 @@ bool sendWithPriority(T& package, std::shared_ptr<U> conn, uint8_t priorityLevel
 
 template <class T, class U>
 bool sendWithPriority(T&& package, std::shared_ptr<U> conn, uint8_t priorityLevel) {
-  painlessmesh::protocol::Variant variant(package);
+  painlessmesh::protocol::Variant variant(&package);
   TSTRING msg;
   variant.printTo(msg);
   return conn->addMessageWithPriority(msg, priorityLevel);