snark13/Xteink-X4-crosspoint-reader
1
0
Fork 0
mirror of https://github.com/daveallie/crosspoint-reader.git synced 2026-02-07 16:17:38 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #4 from swwilshub/claude/fix-wifi-lockup-GwiXb

Browse Source 
Add watchdog resets at upload start without yields
This commit is contained in:
swwilshub 2026-01-13 00:41:54 +00:00 committed by GitHub
commit 61dec47652
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/network/CrossPointWebServer.cpp
View File

@ -338,6 +338,9 @@ static bool flushUploadBuffer() {
void CrossPointWebServer::handleUpload() const { void CrossPointWebServer::handleUpload() const {
static size_t lastLoggedSize = 0; static size_t lastLoggedSize = 0;
// Reset watchdog at start of every upload callback - HTTP parsing can be slow
esp_task_wdt_reset();
// Safety check: ensure server is still valid // Safety check: ensure server is still valid
if (!running || !server) { if (!running || !server) {
Serial.printf("[%lu] [WEB] [UPLOAD] ERROR: handleUpload called but server not running!\n", millis()); Serial.printf("[%lu] [WEB] [UPLOAD] ERROR: handleUpload called but server not running!\n", millis());
@ -347,6 +350,9 @@ void CrossPointWebServer::handleUpload() const {
const HTTPUpload& upload = server->upload(); const HTTPUpload& upload = server->upload();
if (upload.status == UPLOAD_FILE_START) { if (upload.status == UPLOAD_FILE_START) {
// Reset watchdog - this is the critical 1% crash point
esp_task_wdt_reset();
uploadFileName = upload.filename; uploadFileName = upload.filename;
uploadSize = 0; uploadSize = 0;
uploadSuccess = false; uploadSuccess = false;
@ -382,9 +388,11 @@ void CrossPointWebServer::handleUpload() const {
if (!filePath.endsWith("/")) filePath += "/"; if (!filePath.endsWith("/")) filePath += "/";
filePath += uploadFileName; filePath += uploadFileName;
// Check if file already exists // Check if file already exists - SD operations can be slow
esp_task_wdt_reset();
if (SdMan.exists(filePath.c_str())) { if (SdMan.exists(filePath.c_str())) {
Serial.printf("[%lu] [WEB] [UPLOAD] Overwriting existing file: %s\n", millis(), filePath.c_str()); Serial.printf("[%lu] [WEB] [UPLOAD] Overwriting existing file: %s\n", millis(), filePath.c_str());
esp_task_wdt_reset();
SdMan.remove(filePath.c_str()); SdMan.remove(filePath.c_str());
} }