Merge pull request #459 from ESP32Async/fix/close

fix(http): close file before aborting request

Author: mathieucarbou

src/WebHandlers.cpp

@@ -178,8 +178,8 @@ bool AsyncStaticWebHandler::_searchFile(AsyncWebServerRequest *request, const St
     char *_tempPath = (char *)malloc(pathLen + 1);
     if (_tempPath == NULL) {
       async_ws_log_e("Failed to allocate");
-      request->abort();
       request->_tempFile.close();
+      request->abort();
       return false;
     }
     snprintf_P(_tempPath, pathLen + 1, PSTR("%s"), path.c_str());

src/WebRequest.cpp

@@ -279,7 +279,7 @@ void AsyncWebServerRequest::onDisconnect(ArDisconnectHandler fn) {
 }
 
 void AsyncWebServerRequest::_onDisconnect() {
-  // os_printf("d\n");
+  async_ws_log_v("onDisconnect() cb for request: %s", _url.c_str());
   if (_onDisconnectfn) {
     _onDisconnectfn();
   }
@@ -1066,7 +1066,7 @@ void AsyncWebServerRequest::abort() {
     _sent = true;
     _paused = false;
     _this.reset();
-    // async_ws_log_e("AsyncWebServerRequest::abort");
+    async_ws_log_v("Abort request: %s", _url.c_str());
     _client->abort();
   }
 }