Skip to content

Commit 00743b4

Browse files
committed
fix(http): close file before aborting request
1 parent 9ad13e8 commit 00743b4

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/WebHandlers.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,8 +178,8 @@ bool AsyncStaticWebHandler::_searchFile(AsyncWebServerRequest *request, const St
178178
char *_tempPath = (char *)malloc(pathLen + 1);
179179
if (_tempPath == NULL) {
180180
async_ws_log_e("Failed to allocate");
181-
request->abort();
182181
request->_tempFile.close();
182+
request->abort();
183183
return false;
184184
}
185185
snprintf_P(_tempPath, pathLen + 1, PSTR("%s"), path.c_str());

src/WebRequest.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ void AsyncWebServerRequest::onDisconnect(ArDisconnectHandler fn) {
279279
}
280280

281281
void AsyncWebServerRequest::_onDisconnect() {
282-
// os_printf("d\n");
282+
async_ws_log_v("onDisconnect() cb for request: %s", _url.c_str());
283283
if (_onDisconnectfn) {
284284
_onDisconnectfn();
285285
}
@@ -1066,7 +1066,7 @@ void AsyncWebServerRequest::abort() {
10661066
_sent = true;
10671067
_paused = false;
10681068
_this.reset();
1069-
// async_ws_log_e("AsyncWebServerRequest::abort");
1069+
async_ws_log_v("Abort request: %s", _url.c_str());
10701070
_client->abort();
10711071
}
10721072
}

0 commit comments

Comments
 (0)