Config Override RgxState & RgxNAPT, update inline docs

platformio_tasmota_cenv_sample.ini

@@ -20,8 +20,8 @@ build_flags                 = ${esp82xx_defaults.build_flags}
 build_unflags               = ${esp_defaults.build_unflags}
 build_flags                 = ${esp82xx_defaults.build_flags}
                               -D FIRMWARE_RANGE_EXTENDER
-                              -D USE_WIFI_RANGE_EXTENDER
                               -D PIO_FRAMEWORK_ARDUINO_LWIP2_HIGHER_BANDWIDTH
+                              -D USE_WIFI_RANGE_EXTENDER
                               -D USE_WIFI_RANGE_EXTENDER_NAPT
 
 ; *** Tasmota development core version ESP32 IDF3.3.5 / Currently none, same as default
@@ -87,6 +87,13 @@ build_flags                 = ${esp32_defaults.build_flags}
                               -Wno-switch-unreachable
                               ;-DESP32_STAGE=true
 
+[env:tasmota32-rangeextender]
+extends                     = env:tasmota32idf4
+build_flags                 = ${env:tasmota32idf4.build_flags} 
+                              -D FIRMWARE_TASMOTA32
+                              -D USE_WIFI_RANGE_EXTENDER
+                              -D USE_WIFI_RANGE_EXTENDER_NAPT
+
 ; *** Debug version used for PlatformIO Home Project Inspection
 [env:tasmota-debug]
 build_type                  = debug

tasmota/settings.ino

@@ -871,6 +871,8 @@ void SettingsDefaultSet2(void) {
   SettingsUpdateText(SET_HOSTNAME, WIFI_HOSTNAME);
   SettingsUpdateText(SET_RGX_SSID, PSTR(WIFI_RGX_SSID));
   SettingsUpdateText(SET_RGX_PASSWORD, PSTR(WIFI_RGX_PASSWORD));
+  Settings->sbflag1.range_extender = WIFI_RGX_STATE;
+  Settings->sbflag1.range_extender_napt = WIFI_RGX_NAPT;
 
   // Syslog
   SettingsUpdateText(SET_SYSLOG_HOST, PSTR(SYS_LOG_HOST));
@@ -1386,6 +1388,8 @@ void SettingsDelta(void) {
       ParseIPv4(&Settings->ipv4_address[4], PSTR(WIFI_DNS2));
     }
     if (Settings->version < 0x09050005) {
+      Settings->sbflag1.range_extender = WIFI_RGX_STATE;
+      Settings->sbflag1.range_extender_napt = WIFI_RGX_NAPT;
       ParseIPv4(&Settings->ipv4_rgx_address, PSTR(WIFI_RGX_IP_ADDRESS));
       ParseIPv4(&Settings->ipv4_rgx_subnetmask, PSTR(WIFI_RGX_SUBNETMASK));
       SettingsUpdateText(SET_RGX_SSID, PSTR(WIFI_RGX_SSID));

tasmota/tasmota_globals.h

@@ -344,6 +344,12 @@ String EthernetMacAddress(void);
 #define STARTING_OFFSET             30         // NOVA SDS parameter used in settings
 #endif
 
+#ifndef WIFI_RGX_STATE
+#define WIFI_RGX_STATE              0
+#endif
+#ifndef WIFI_RGX_NAPT
+#define WIFI_RGX_NAPT               0
+#endif
 #ifndef WIFI_RGX_SSID
 #define WIFI_RGX_SSID               ""
 #endif

tasmota/xdrv_58_range_extender.ino

@@ -22,24 +22,34 @@
 To use this, add the following to your user_config_override.h
 #define USE_WIFI_RANGE_EXTENDER
 
-Additionally PIO_FRAMEWORK_ARDUINO_LWIP2_HIGHER_BANDWIDTH must be set in your build options.
+Additionally, for the ESP8266, PIO_FRAMEWORK_ARDUINO_LWIP2_HIGHER_BANDWIDTH must be 
+set in your build options.
 For example, in your platfromio_tasmota_cenv.ini, you will need an entry such as:
 [env:tasmota-rangeextender]
 build_flags = ${common.build_flags}
-              -D FIRMWARE_RANGE_EXTENDER
               -D PIO_FRAMEWORK_ARDUINO_LWIP2_HIGHER_BANDWIDTH
 
+For the ESP32, the arduino-esp32 library must be at least version 2, with 
+CONFIG_LWIP_IP_FORWARD option set, and optionally CONFIG_LWIP_IPV4_NAPT.
+
 If you want to support NAPT (removing the need for routes on a core router):
 #define USE_WIFI_RANGE_EXTENDER_NAPT
 
+
 An example full static configuration:
 #define USE_WIFI_RANGE_EXTENDER
 #define USE_WIFI_RANGE_EXTENDER_NAPT
+#define WIFI_RGX_STATE 1
+#define WIFI_RGX_NAPT 1
 #define WIFI_RGX_SSID "rangeextender"
 #define WIFI_RGX_PASSWORD "securepassword"
 #define WIFI_RGX_IP_ADDRESS "10.99.1.1"
 #define WIFI_RGX_SUBNETMASK "255.255.255.0"
 
+
+A full command to enable the Range Extender, including with NAPT could be:
+Backlog RgxSSID rangeextender ; RgxPassword securepassword ; RgxAddress 192.168.123.1 ; RgxSubnet 255.255.255.0; RgxState 1 ; RgxNAPT 1
+
 \*********************************************************************************************/
 
 #define XDRV_58 58
@@ -76,10 +86,8 @@ An example full static configuration:
 
 const char kDrvRgxCommands[] PROGMEM = "Rgx|" // Prefix
                                        "State"
-                                       "|"
-                                       D_CMND_SSID
-                                       "|"
-                                       D_CMND_PASSWORD
+                                       "|" D_CMND_SSID
+                                       "|" D_CMND_PASSWORD
 #ifdef USE_WIFI_RANGE_EXTENDER_NAPT
                                        "|"
                                        "NAPT"
@@ -122,13 +130,11 @@ void (*const DrvRgxCommand[])(void) PROGMEM = {
 #define RGX_CONFIG_INCOMPLETE 3
 #define RGX_SETUP_NAPT 4
 
-typedef struct {
+typedef struct
+{
   uint8_t status = RGX_NOT_CONFIGURED;
 #ifdef USE_WIFI_RANGE_EXTENDER_NAPT
   bool napt_enabled = false;
-#ifdef ESP8266
-  bool napt_inited = false;
-#endif  // ESP8266
 #endif // USE_WIFI_RANGE_EXTENDER_NAPT
 } TRgxSettings;
 
@@ -154,11 +160,15 @@ void RgxCheckConfig(void)
   }
 }
 
-void CmndRgxState(void) {
-  if ((XdrvMailbox.payload >= 0) && (XdrvMailbox.payload <= 1)) {
-    if (Settings->sbflag1.range_extender != XdrvMailbox.payload) {
+void CmndRgxState(void)
+{
+  if ((XdrvMailbox.payload >= 0) && (XdrvMailbox.payload <= 1))
+  {
+    if (Settings->sbflag1.range_extender != XdrvMailbox.payload)
+    {
       Settings->sbflag1.range_extender = XdrvMailbox.payload;
-      if (0 == XdrvMailbox.payload) {  // Turn off
+      if (0 == XdrvMailbox.payload)
+      { // Turn off
         TasmotaGlobal.restart_flag = 2;
       }
     }
@@ -166,14 +176,19 @@ void CmndRgxState(void) {
   ResponseCmndStateText(Settings->sbflag1.range_extender);
 }
 
-void CmndRgxAddresses(void) {
+void CmndRgxAddresses(void)
+{
   char network_address[22];
   ext_snprintf_P(network_address, sizeof(network_address), PSTR(" (%_I)"), (uint32_t)NetworkAddress());
   uint32_t ipv4_address;
-  if (ParseIPv4(&ipv4_address, XdrvMailbox.data)) {
-    if (XdrvMailbox.command[3] == 'S') {      // Subnet
+  if (ParseIPv4(&ipv4_address, XdrvMailbox.data))
+  {
+    if (XdrvMailbox.command[3] == 'S') // Subnet
+    {
       Settings->ipv4_rgx_subnetmask = ipv4_address;
-    } else {
+    }
+    else
+    {
       Settings->ipv4_rgx_address = ipv4_address;
     }
     RgxSettings.status = RGX_FORCE_CONFIGURE;
@@ -181,30 +196,46 @@ void CmndRgxAddresses(void) {
   ResponseRgxConfig();
 }
 
-void CmndRgxSSID(void) {
-  if (XdrvMailbox.data_len > 0) {
+void CmndRgxSSID(void)
+{
+  if (XdrvMailbox.data_len > 0)
+  {
     SettingsUpdateText(SET_RGX_SSID, (SC_CLEAR == Shortcut()) ? "" : XdrvMailbox.data);
-  }
     RgxSettings.status = RGX_FORCE_CONFIGURE;
+  }
   ResponseRgxConfig();
 }
 
-void CmndRgxPassword(void) {
-  if (XdrvMailbox.data_len > 0) {
+void CmndRgxPassword(void)
+{
+  if (XdrvMailbox.data_len > 0)
+  {
     SettingsUpdateText(SET_RGX_PASSWORD, (SC_CLEAR == Shortcut()) ? "" : XdrvMailbox.data);
-  }
     RgxSettings.status = RGX_FORCE_CONFIGURE;
+  }
   ResponseRgxConfig();
 }
 
 #ifdef USE_WIFI_RANGE_EXTENDER_NAPT
-void CmndRgxNAPT(void) {
-  if ((XdrvMailbox.payload >= 0) && (XdrvMailbox.payload <= 1)) {
-    if (Settings->sbflag1.range_extender_napt != XdrvMailbox.payload) {
+void CmndRgxNAPT(void)
+{
+  if ((XdrvMailbox.payload >= 0) && (XdrvMailbox.payload <= 1))
+  {
+    if (Settings->sbflag1.range_extender_napt != XdrvMailbox.payload)
+    {
       Settings->sbflag1.range_extender_napt = XdrvMailbox.payload;
+      if (0 == XdrvMailbox.payload)
+      { // Turn off
+        // ESP32 does not disable reliably, reboot to ensure a complete disable of NAPT
+        // rebooting also completely frees up the used RAM
+        TasmotaGlobal.restart_flag = 2;
+      }
+      else
+      {
         RgxSettings.status = RGX_FORCE_CONFIGURE;
       }
     }
+  }
   ResponseCmndStateText(Settings->sbflag1.range_extender_napt);
 };
 #endif // USE_WIFI_RANGE_EXTENDER_NAPT
@@ -247,31 +278,36 @@ void rngxSetup()
   // WiFi.softAPConfig(EXTENDER_LOCAL_IP, EXTENDER_GATEWAY_IP, EXTENDER_SUBNET);
   WiFi.softAPConfig(Settings->ipv4_rgx_address, Settings->ipv4_rgx_address, Settings->ipv4_rgx_subnetmask);
   WiFi.softAP(SettingsText(SET_RGX_SSID), SettingsText(SET_RGX_PASSWORD));
-  AddLog(LOG_LEVEL_INFO, PSTR("RGX: WiFi Extender AP Enabled"));
+  AddLog(LOG_LEVEL_INFO, PSTR("RGX: WiFi Extender AP Enabled with SSID: %s"), WiFi.softAPSSID().c_str());
   RgxSettings.status = RGX_SETUP_NAPT;
 }
 
-void rngxSetupNAPT(void) {
-  // A short delay is required for enabling NAPT to work on the ESP32
+void rngxSetupNAPT(void)
+{
+  // A short delay is required for enabling NAPT to work on the ESP32, hence a dedicated
+  // function called a second later
 #ifdef USE_WIFI_RANGE_EXTENDER_NAPT
-  if (Settings->sbflag1.range_extender_napt && !RgxSettings.napt_enabled) {
+  if (Settings->sbflag1.range_extender_napt && !RgxSettings.napt_enabled)
+  {
 #ifdef ESP8266
-    if (!RgxSettings.napt_inited) {
+    // ip_napt_init can only be called once, however device will reboot when disabled
+    // so no need to limit calls to init separately.
     err_t ret = ip_napt_init(NAPT, NAPT_PORT);
-      if (ret == ERR_OK) {
+    if (ret == ERR_OK)
+    {
       AddLog(LOG_LEVEL_INFO, PSTR("RGX: NAPT initialization complete"));
-        RgxSettings.napt_inited = true;
-      } else {
-        AddLog(LOG_LEVEL_ERROR, PSTR("RGX: NAPT initialization failed! (%d)"), ret);
-      }
-    }
-    if (RgxSettings.napt_inited) {
       err_t ret = ip_napt_enable_no(SOFTAP_IF, 1);
-      if (ret == ERR_OK) {
+      if (ret == ERR_OK)
+      {
         AddLog(LOG_LEVEL_INFO, PSTR("RGX: NAPT Enabled"));
         RgxSettings.napt_enabled = true;
       }
     }
+    else
+    {
+      AddLog(LOG_LEVEL_ERROR, PSTR("RGX: NAPT initialization failed! (%d)"), ret);
+    }
+
 #endif // ESP8266
 #ifdef ESP32
     ip_napt_enable(WiFi.softAPIP(), 1);
@@ -279,10 +315,15 @@ void rngxSetupNAPT(void) {
     RgxSettings.napt_enabled = true;
 #endif // ESP32
   }
-  else if (!Settings->sbflag1.range_extender_napt && RgxSettings.napt_enabled) {
+  // This code path is no longer used as device will reboot to disable NAPT, maybe
+  // restored when working correctly on ESP32
+  /*
+  else if (!Settings->sbflag1.range_extender_napt && RgxSettings.napt_enabled)
+  {
 #ifdef ESP8266
     err_t ret = ip_napt_enable_no(SOFTAP_IF, 0);
-    if (ret == ERR_OK) {
+    if (ret == ERR_OK)
+    {
       AddLog(LOG_LEVEL_INFO, "RGX: NAPT Disabled");
       RgxSettings.napt_enabled = false;
     }
@@ -292,6 +333,7 @@ void rngxSetupNAPT(void) {
     AddLog(LOG_LEVEL_INFO, "RGX: NAPT Disabled, reboot maybe required");
 #endif // ESP32
   }
+  */
 #endif // USE_WIFI_RANGE_EXTENDER_NAPT
   RgxSettings.status = RGX_CONFIGURED;
 }
@@ -300,13 +342,16 @@ void rngxSetupNAPT(void) {
  * Interface
 \*********************************************************************************************/
 
-bool Xdrv58(uint8_t function) {
+bool Xdrv58(uint8_t function)
+{
   bool result = false;
 
-  if (FUNC_COMMAND == function) {
+  if (FUNC_COMMAND == function)
+  {
     result = DecodeCommand(kDrvRgxCommands, DrvRgxCommand);
   }
-  else if (Settings->sbflag1.range_extender && !TasmotaGlobal.restart_flag) {
+  else if (Settings->sbflag1.range_extender && !TasmotaGlobal.restart_flag)
+  {
     switch (function)
     {
     case FUNC_PRE_INIT:
@@ -332,6 +377,7 @@ bool Xdrv58(uint8_t function) {
       }
       else if (RgxSettings.status == RGX_SETUP_NAPT)
       {
+        // Call NAPT a second later as ESP32 requires short delay
         rngxSetupNAPT();
       }
       else if (RgxSettings.status == RGX_CONFIGURED)