Use ESP.reset() rather than ESP.restart() - due to exceptions encountered on esp8266

interface/src/constants/Endpoints.js

@@ -13,4 +13,4 @@ export const SYSTEM_STATUS_ENDPOINT = ENDPOINT_ROOT + "systemStatus";
 export const SIGN_IN_ENDPOINT = ENDPOINT_ROOT + "signIn";
 export const VERIFY_AUTHORIZATION_ENDPOINT = ENDPOINT_ROOT + "verifyAuthorization";
 export const SECURITY_SETTINGS_ENDPOINT = ENDPOINT_ROOT + "securitySettings";
-export const RESTART_ENDPOINT = ENDPOINT_ROOT + "restart";
+export const RESET_ENDPOINT = ENDPOINT_ROOT + "reset";

interface/src/containers/SystemStatus.js

@@ -22,7 +22,7 @@ import DataUsageIcon from '@material-ui/icons/DataUsage';
 import AutorenewIcon from '@material-ui/icons/Autorenew';
 import RefreshIcon from '@material-ui/icons/Refresh';
 
-import { SYSTEM_STATUS_ENDPOINT, RESTART_ENDPOINT } from '../constants/Endpoints';
+import { SYSTEM_STATUS_ENDPOINT, RESET_ENDPOINT } from '../constants/Endpoints';
 import { restComponent } from '../components/RestComponent';
 import LoadingNotification from '../components/LoadingNotification';
 import SectionContent from '../components/SectionContent';
@@ -42,7 +42,7 @@ class SystemStatus extends Component {
     super(props);
 
     this.state = {
-      confirmRestart: false,
+      confirmReset: false,
       processing: false
     }
   }
@@ -112,53 +112,53 @@ class SystemStatus extends Component {
         <Button startIcon={<RefreshIcon />} variant="contained" color="secondary" className={classes.button} onClick={this.props.loadData}>
           Refresh
         </Button>
-        <Button startIcon={<AutorenewIcon />} variant="contained" color="secondary" className={classes.button} onClick={this.onRestart}>
+        <Button startIcon={<AutorenewIcon />} variant="contained" color="secondary" className={classes.button} onClick={this.onReset}>
-          Restart
+          Reset
         </Button>
       </div>
     );
   }
 
-  onRestart = () => {
+  onReset = () => {
-    this.setState({ confirmRestart: true });
+    this.setState({ confirmReset: true });
   }
 
-  onRestartRejected = () => {
+  onResetRejected = () => {
-    this.setState({ confirmRestart: false });
+    this.setState({ confirmReset: false });
   }
 
-  onRestartConfirmed = () => {
+  onResetConfirmed = () => {
     this.setState({ processing: true });
-    redirectingAuthorizedFetch(RESTART_ENDPOINT, { method: 'POST' })
+    redirectingAuthorizedFetch(RESET_ENDPOINT, { method: 'POST' })
       .then(response => {
         if (response.status === 200) {
-          this.props.enqueueSnackbar("Device is restarting", { variant: 'info' });
+          this.props.enqueueSnackbar("Device is reseting", { variant: 'info' });
-          this.setState({ processing: false, confirmRestart: false });
+          this.setState({ processing: false, confirmReset: false });
         } else {
           throw Error("Invalid status code: " + response.status);
         }
       })
       .catch(error => {
-        this.props.enqueueSnackbar(error.message || "Problem restarting device", { variant: 'error' });
+        this.props.enqueueSnackbar(error.message || "Problem resetting device", { variant: 'error' });
-        this.setState({ processing: false, confirmRestart: false });
+        this.setState({ processing: false, confirmReset: false });
       });
   }
 
-  renderRestartDialog() {
+  renderResetDialog() {
     return (
       <Dialog
-        open={this.state.confirmRestart}
+        open={this.state.confirmReset}
-        onClose={this.onRestartRejected}
+        onClose={this.onResetRejected}
       >
-        <DialogTitle>Confirm Restart</DialogTitle>
+        <DialogTitle>Confirm Reset</DialogTitle>
         <DialogContent dividers={true}>
-          Are you sure you want to restart the device?
+          Are you sure you want to reset the device?
         </DialogContent>
         <DialogActions>
-          <Button startIcon={<AutorenewIcon />} variant="contained" onClick={this.onRestartConfirmed} disabled={this.state.processing} color="primary" autoFocus>
+          <Button startIcon={<AutorenewIcon />} variant="contained" onClick={this.onResetConfirmed} disabled={this.state.processing} color="primary" autoFocus>
-            Restart
+            Reset
           </Button>
-          <Button variant="contained" onClick={this.onRestartRejected} color="secondary">
+          <Button variant="contained" onClick={this.onResetRejected} color="secondary">
             Cancel
           </Button>
         </DialogActions>
@@ -178,7 +178,7 @@ class SystemStatus extends Component {
             () => this.renderSystemStatus(data, classes)
           }
         />
-        {this.renderRestartDialog()}
+        {this.renderResetDialog()}
       </SectionContent>
     )
   }

lib/framework/ESP8266React.cpp

@@ -6,7 +6,7 @@ ESP8266React::ESP8266React(AsyncWebServer* server, FS* fs):
   _apSettingsService(server, fs, &_securitySettingsService),
   _ntpSettingsService(server, fs, &_securitySettingsService),
   _otaSettingsService(server, fs, &_securitySettingsService),
-  _restartService(server, &_securitySettingsService),
+  _ResetService(server, &_securitySettingsService),
   _authenticationService(server, &_securitySettingsService),
   _wifiScanner(server, &_securitySettingsService),
   _wifiStatus(server, &_securitySettingsService),

lib/framework/ESP8266React.h

@@ -24,7 +24,7 @@
 #include <NTPStatus.h>
 #include <APStatus.h>
 #include <SystemStatus.h>
-#include <RestartService.h>
+#include <ResetService.h>
 
 class ESP8266React {
 
@@ -47,7 +47,7 @@ class ESP8266React {
     APSettingsService _apSettingsService;
     NTPSettingsService _ntpSettingsService;
     OTASettingsService _otaSettingsService;
-    RestartService _restartService;    
+    ResetService _ResetService;    
     AuthenticationService _authenticationService;
 
     

lib/framework/ResetService.cpp

@@ -0,0 +1,14 @@
+#include <ResetService.h>
+
+ResetService::ResetService(AsyncWebServer* server, SecurityManager* securityManager) {
+  server->on(RESET_SERVICE_PATH, HTTP_POST,
+    securityManager->wrapRequest(std::bind(&ResetService::reset, this, std::placeholders::_1), AuthenticationPredicates::IS_ADMIN)
+  );
+}
+
+void ResetService::reset(AsyncWebServerRequest *request) {  
+  request->onDisconnect([](){
+    ESP.reset();
+  });
+  request->send(200);
+}

lib/framework/ResetService.h

@@ -0,0 +1,29 @@
+#ifndef ResetService_h
+#define ResetService_h
+
+#if defined(ESP8266)
+  #include <ESP8266WiFi.h>
+  #include <ESPAsyncTCP.h>
+#elif defined(ESP_PLATFORM)
+  #include <WiFi.h>
+  #include <AsyncTCP.h>
+#endif
+
+#include <ESPAsyncWebServer.h>
+#include <SecurityManager.h>
+
+#define RESET_SERVICE_PATH "/rest/reset"
+
+class ResetService {
+
+  public:
+   
+    ResetService(AsyncWebServer* server, SecurityManager* securityManager);
+
+  private:
+
+    void reset(AsyncWebServerRequest *request);
+
+};
+
+#endif // end ResetService_h

lib/framework/RestartService.cpp

@@ -1,14 +0,0 @@
-#include <RestartService.h>
-
-RestartService::RestartService(AsyncWebServer* server, SecurityManager* securityManager) {
-  server->on(RESTART_SERVICE_PATH, HTTP_POST,
-    securityManager->wrapRequest(std::bind(&RestartService::restart, this, std::placeholders::_1), AuthenticationPredicates::IS_ADMIN)
-  );
-}
-
-void RestartService::restart(AsyncWebServerRequest *request) {  
-  request->onDisconnect([](){
-    ESP.restart();
-  });
-  request->send(200);
-}

lib/framework/RestartService.h

@@ -1,29 +0,0 @@
-#ifndef RestartService_h
-#define RestartService_h
-
-#if defined(ESP8266)
-  #include <ESP8266WiFi.h>
-  #include <ESPAsyncTCP.h>
-#elif defined(ESP_PLATFORM)
-  #include <WiFi.h>
-  #include <AsyncTCP.h>
-#endif
-
-#include <ESPAsyncWebServer.h>
-#include <SecurityManager.h>
-
-#define RESTART_SERVICE_PATH "/rest/restart"
-
-class RestartService {
-
-  public:
-   
-    RestartService(AsyncWebServer* server, SecurityManager* securityManager);
-
-  private:
-
-    void restart(AsyncWebServerRequest *request);
-
-};
-
-#endif // end RestartService_h