Merge pull request #62 from theelims/task-isolation-restart-sleep

Isolate non-returning functions in new tasks
theelims · Dec 20, 2024 · f44de68 · f44de68
2 parents 86e165f + 7140b48
commit f44de68
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 8 deletions.
diff --git a/lib/framework/RestartService.h b/lib/framework/RestartService.h
@@ -17,6 +17,7 @@
 
 #include <WiFi.h>
 
+#include <ESPmDNS.h>
 #include <PsychicHttp.h>
 #include <SecurityManager.h>
 
@@ -31,9 +32,16 @@ class RestartService
 
     static void restartNow()
     {
-        WiFi.disconnect(true);
-        delay(500);
-        ESP.restart();
+        xTaskCreate(
+            [](void *pvParams) {
+                delay(250);
+                MDNS.end();
+                delay(100);
+                WiFi.disconnect(true);
+                delay(500);
+                ESP.restart();
+            },
+            "Restart task", 4096, nullptr, 10, nullptr);
     }
 
 private:

diff --git a/lib/framework/SleepService.cpp b/lib/framework/SleepService.cpp
@@ -86,9 +86,10 @@ void SleepService::sleepNow()
     Serial.println("Good by!");
 #endif
 
-    // Just to be sure
-    delay(100);
-
-    // Hibernate
-    esp_deep_sleep_start();
+    xTaskCreate(
+        [](void *pvParams) {
+            delay(200);
+            esp_deep_sleep_start();
+        },
+        "Sleep task", 4096, nullptr, 10, nullptr);
 }