OTA_update_STM32_using_ESP32 icon indicating copy to clipboard operation
OTA_update_STM32_using_ESP32 copied to clipboard

Bugfix/maxou/fix timeout serial

Open pinkymaxou opened this issue 2 years ago • 1 comments

Bug fix, the delay doesn't work

There were a bug where the timeout is not respected, causing it to not work in some situation. This line: vTaskDelay(1 / portTICK_PERIOD_MS); If FreeRTOS is set lower than 1000 HZ (1 ms per tick) it cause the calculation to return 0 so it doesn't have any meaningful delay. Now it does with this fix. If FreeRTOS is set a 100 HZ (10 ms per tick) with 1000 ms timeout, the loop will operate 100 times and create 100x10ms delay.

See: https://github.com/ESP32-Musings/OTA_update_STM32_using_ESP32/issues/4

Prescribed delay is not sufficient

I needed more than 8s to erase flash memory on my microcontroller so 5s is insufficient. Especially if the micro is not a new one.

Header too big to parse

Some modern browser have a tendency to produce ridiculously big HTTP GET header, I increased the webserver buffer size to repair this.

pinkymaxou avatar Mar 14 '23 11:03 pinkymaxou

Can you please accept it so I can put this behind me :(

pinkymaxou avatar Sep 10 '23 19:09 pinkymaxou