-
Notifications
You must be signed in to change notification settings - Fork 7.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
simple_ota_example fails after upgrade to idf v5.1.2 from v 5.1.1 (IDFGH-12486) #13497
Closed
3 tasks done
Labels
Resolution: NA
Issue resolution is unavailable
Status: Done
Issue is done internally
Type: Bug
bugs in IDF
Comments
github-actions
bot
changed the title
simple_ota_example fails after upgrade to idf v5.1.2 from v 5.1.1
simple_ota_example fails after upgrade to idf v5.1.2 from v 5.1.1 (IDFGH-12486)
Mar 29, 2024
espressif-bot
added
Status: In Progress
Work is in progress
and removed
Status: Opened
Issue is new
labels
Apr 1, 2024
espressif-bot
added
Status: Done
Issue is done internally
Resolution: NA
Issue resolution is unavailable
and removed
Status: In Progress
Work is in progress
labels
Apr 1, 2024
Hi, reverting that commit allows the OTA to work in 5.1.2 How do I progress to 5.2.1 which exhibits the same problem? |
Reverting that commit also resolves the problem for me in v5.2.1 Do you know when the fix is likely to appear in the release branches? |
We have backported the fix to release branches as well. The fix should be available soon. |
thanks |
espressif-bot
pushed a commit
that referenced
this issue
Apr 4, 2024
…s received" This reverts commit 04ac8e4. When `http_on_header_event` is called, event_handler is invoked and then the current header key and header value are freed. In the previous approach, `http_on_header_event` was called from `http_on_header_value`, but it lead to an issue where if the value is received in multiple chunks, then the current header key and value were freed and thus header was not processed correctly which might result in connection issues. Calling `http_on_heaher_event` from `http_on_header_field` ensures that the current header field and value are processed properly Fixes #13497 Fixes #13097
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Resolution: NA
Issue resolution is unavailable
Status: Done
Issue is done internally
Type: Bug
bugs in IDF
Answers checklist.
IDF version.
v5.1.2
Espressif SoC revision.
esp32-s3
Operating System used.
Windows
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
CMD
Development Kit.
Custom Board based on esp32-s3 WROOM-1 N16R8
Power Supply used.
USB
What is the expected behavior?
Perform OTA update from github
What is the actual behavior?
Fails with following output
Steps to reproduce.
idf.py set-target esp32-s3
idf.py menuconfig
to configure wifiusing idf 5.1.1
idf.py -p COMxx fullclean build flash monitor
using idf 5.1.2
idf.py -p COMxx fullclean build flash monitor
Debug Logs.
with idf 5.1.1
with idf 5.1.2
The text was updated successfully, but these errors were encountered: