Fixed that issue with invalid pointer. It turned out that just need to copy the data

Author: dyrkin

.gitignore

@@ -1,3 +1,4 @@
+.idea
 .pio
 .vscode/.browse.c_cpp.db*
 .vscode/c_cpp_properties.json

platformio.ini

@@ -14,6 +14,8 @@ board = nodemcuv2
 board_build.filesystem = littlefs
 framework = arduino
 monitor_speed = 115200
+monitor_filters = esp8266_exception_decoder
+build_type = debug
 lib_deps = 
 	me-no-dev/ESP Async WebServer@^1.2.3
 	me-no-dev/ESPAsyncTCP@^1.2.2

src/main.cpp

@@ -7,15 +7,16 @@
 #include "metrics.h"
 
 Settings settings;
-UI ui{80, &settings};
 Temp temp{20};
 Heater heater{&settings, &temp};
 WiFiNetwork wifi{&settings};
+UI ui{80, &settings, &temp, &heater, &wifi};
 Metrics metrics{&settings, &temp, &heater};
 
 void setup()
 {
   Serial.begin(115200);
+  delay(1000);
   LittleFS.begin();
   settings.read();
   configTzTime("CET-1CEST", "pool.ntp.org", "time.nis.gov");

src/metrics.cpp

@@ -27,6 +27,7 @@ void Metrics::send()
         pointDevice.addField("two_sensor_avg_temp", _temp->getAvgTemp());
         pointDevice.addField("sht_humidity", _temp->getShtHumidity());
         pointDevice.addField("heating", _heater->heating());
+        pointDevice.addField("threshold", _settings->temperatureThreshold());
 
         _client->writePoint(pointDevice);
     }

src/settings.cpp

@@ -35,10 +35,11 @@ bool Settings::read()
         return false;
     }
 
-    _ssid = doc[WIFI_SSID_KEY];
-    _password = doc[WIFI_PASSWORD_KEY];
-    _influxdbUrl = doc[INFLUXDB_URL_KEY];
-    _influxdbDatabaseName = doc[INFLUXDB_DATABASE_NAME_KEY];
+    _ssid = strdup(doc[WIFI_SSID_KEY]);
+    _password = strdup(doc[WIFI_PASSWORD_KEY]);
+    _influxdbUrl = strdup(doc[INFLUXDB_URL_KEY]);
+    _influxdbDatabaseName = strdup(doc[INFLUXDB_DATABASE_NAME_KEY]);
+
     _temperatureThreshold = doc[TEMPERATURE_THRESHOLD];
     _heaterEnabled = doc[HEATER_ENABLED];
 
@@ -73,7 +74,7 @@ const char *Settings::ssid()
 
 void Settings::setSsid(const char *ssid)
 {
-    _ssid = ssid;
+    _ssid = strdup(ssid);
 }
 
 const char *Settings::password()
@@ -83,7 +84,7 @@ const char *Settings::password()
 
 void Settings::setPassword(const char *password)
 {
-    _password = password;
+    _password = strdup(password);
 }
 
 const char *Settings::influxdbUrl()
@@ -93,7 +94,7 @@ const char *Settings::influxdbUrl()
 
 void Settings::setInfluxdbUrl(const char *influxdbUrl)
 {
-    _influxdbUrl = influxdbUrl;
+    _influxdbUrl = strdup(influxdbUrl);
 }
 
 const char *Settings::influxdbDatabaseName()
@@ -103,7 +104,7 @@ const char *Settings::influxdbDatabaseName()
 
 void Settings::setInfluxdbDatabaseName(const char *influxdbDatabaseName)
 {
-    _influxdbDatabaseName = influxdbDatabaseName;
+    _influxdbDatabaseName = strdup(influxdbDatabaseName);
 }
 
 float Settings::temperatureThreshold()

src/ui.cpp

@@ -5,10 +5,15 @@
 #include <ESPAsyncWebServer.h>
 #include "AsyncJson.h"
 #include "ArduinoJson.h"
-#include "settings.h"
+#include <Ticker.h>
+#include "functional"
 
-UI::UI(int port, Settings *settings) : _server(port) {
+UI::UI(int port, Settings *settings, Temp *temp, Heater *heater, WiFiNetwork *wifiNetwork) : _server(port)
+{
     _settings = settings;
+    _temp = temp;
+    _heater = heater;
+    _wifiNetwork = wifiNetwork;
 }
 
 void UI::begin()
@@ -28,23 +33,55 @@ void UI::begin()
         doc["ssid"] = _settings->ssid();
         doc["influxdb_url"] = _settings->influxdbUrl();
         doc["influxdb_db_name"] = _settings->influxdbDatabaseName();
+        doc["heater_enabled"] = _settings->heaterEnabled();
 
         serializeJson(doc, *response);
 
         request->send(response);
     });
 
+    _server.on(
+        "/settings", HTTP_POST, [](AsyncWebServerRequest *request) {}, NULL,
+        [&](AsyncWebServerRequest *request, uint8_t *data, size_t len, size_t index, size_t total) {
+            DynamicJsonDocument doc(200);
+            auto error = deserializeJson(doc, ((const char *)data));
+
+            if (error)
+            {
+                Serial.println("Failed to parse config file");
+            }
+
+            _settings->setSsid(doc["ssid"]);
+            _settings->setPassword(doc["password"]);
+            _settings->setInfluxdbUrl(doc["influxdb_url"]);
+            _settings->setInfluxdbDatabaseName(doc["influxdb_db_name"]);
+            _settings->setHeaterEnabled(doc["heater_enabled"]);
+            request->send(200);
+            // Ticker tick;
+            // tick.once_ms_scheduled(1, [&]() {
+            //     _wifiNetwork->begin();
+            // });
+        });
+
     _server.on("/state", HTTP_GET, [&](AsyncWebServerRequest *request) {
         AsyncResponseStream *response = request->beginResponseStream("application/json");
         DynamicJsonDocument doc(200);
-        doc["ssid"] = _settings->ssid();
-        doc["influxdb_url"] = _settings->influxdbUrl();
-        doc["influxdb_db_name"] = _settings->influxdbDatabaseName();
+        doc["heating"] = _heater->heating();
+        doc["avg_temp"] = _temp->getAvgTemp();
+        doc["mcp_temp"] = _temp->getMcpTemp();
+        doc["sht_temp"] = _temp->getShtTemp();
+        doc["sht_humidity"] = _temp->getShtHumidity();
+        doc["heater_enabled"] = _settings->heaterEnabled();
 
         serializeJson(doc, *response);
 
         request->send(response);
     });
 
     _server.begin();
+}
+
+void UI::onSettingsUpdated()
+{
+    _wifiNetwork->begin();
 }

src/ui.h

@@ -1,14 +1,21 @@
 #include <ESPAsyncWebServer.h>
 #include "settings.h"
+#include "temp.h"
+#include "heater.h"
+#include "wifiNetwork.h"
 
 class UI
 {
 public:
-    UI(int port, Settings *settings);
+    UI(int port, Settings *settings, Temp *temp, Heater *heater, WiFiNetwork *wifiNetwork);
     void begin();
 
 private:
     int _port;
     AsyncWebServer _server;
     Settings *_settings;
+    Temp *_temp;
+    Heater *_heater;
+    WiFiNetwork *_wifiNetwork;
+    void onSettingsUpdated();
 };

src/wifiNetwork.h

@@ -1,3 +1,6 @@
+#ifndef WIFI_NETWORK_H
+#define WIFI_NETWORK_H
+
 #include <Arduino.h>
 #include "settings.h"
 
@@ -10,4 +13,8 @@ class WiFiNetwork
 
 private:
     Settings *_settings;
+    void connect(const char *ssid, const char *password);
+    bool softAP;
 };
+
+#endif

src/wiifiNetwork.cpp

@@ -1,21 +1,59 @@
 #include "wifiNetwork.h"
 #include <ESP8266WiFi.h>
 
+const char *_softAPSsid = "thermobox";
+const char *_softAPPassword = "thermobox";
+
 WiFiNetwork::WiFiNetwork(Settings *settings)
 {
   _settings = settings;
 }
 
 void WiFiNetwork::begin()
 {
-  WiFi.begin(_settings->ssid(), _settings->password());
-  while (WiFi.status() != WL_CONNECTED)
+  connect(_settings->ssid(), _settings->password());
+}
+
+void WiFiNetwork::connect(const char *ssid, const char *password)
+{
+  WiFi.disconnect();
+  WiFi.softAPdisconnect();
+  if (!ssid || *ssid == 0x00 || strlen(ssid) == 0)
   {
-    delay(1000);
-    Serial.println("Connecting to WiFi..");
+    WiFi.softAP(_softAPSsid, _softAPPassword);
+    Serial.println("Created Soft AP");
+    softAP = true;
+  }
+  else
+  {
+    int attempts = 20;
+    WiFi.begin(ssid, password);
+
+    while (WiFi.status() != WL_CONNECTED && attempts >= 0)
+    {
+      attempts--;
+      delay(1000);
+      Serial.println("Connecting to WiFi..");
+    }
+
+    if (WiFi.status() != WL_CONNECTED)
+    {
+      Serial.println("Can't connect to WiFi");
+      connect(NULL, NULL);
+    }
+    else
+    {
+      softAP = false;
+      Serial.println("Connected to WiFi");
+    }
   }
 }
 
-String WiFiNetwork::ip() {
+String WiFiNetwork::ip()
+{
+  if (softAP)
+  {
+    return WiFi.softAPIP().toString();
+  }
   return WiFi.localIP().toString();
 }