Skip to content
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

V5.4 (IDFGH-14430) #15210

Closed
wants to merge 685 commits into from
Closed

V5.4 (IDFGH-14430) #15210

wants to merge 685 commits into from

Conversation

Gui71
Copy link

@Gui71 Gui71 commented Jan 15, 2025

Description

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

zhaoweiliang2021 and others added 30 commits December 16, 2024 11:03
This commit adds support to handle HTTP/1.0 requests alongside HTTP/1.1 for
legacy compliance purposes.
Most esp_netif public API check for invalid arguments, but when enabling
PPP the macros to check netif type could potentially dereference
esp_netif without any null-check.

Releted to espressif#14816
- Check Access Address when receive connection request PDU
- Fix issue with BLE5.0 duplicate scan for chained packets
Delete the secondary judgment of whether it is the root node in the sending task

Closes espressif#14373

Signed-off-by: zhangyanjiao <[email protected]>
…peration

(cherry picked from commit 5acb9c9)

Co-authored-by: zhanghaipeng <[email protected]>
ci(docs): revert d98e77a to simplify CI (v5.4)

See merge request espressif/esp-idf!35684
This way the setuptools version dependency resolution will be done by
the upgraded pip.
PDF build would randomly fail after S3 docs grew to a certain size.

At 3k+ pages PDF is not very usable anyways.
For offline use an HTML archive makes more sense.
submodule update: git log --oneline a587d929..fa4dffdf
  - autoip: Choose next address after rate limit (espressif/esp-lwip@fa4dffdfa)
The active.py script is currently clearing the IDF_PYTHON_ENV_PATH,
preventing it from being set to a custom location for the python virtual
environment directory. Although the install script checks to ensure that
an existing python virtual environment is not overwritten with one for a
different ESP-IDF version than it was originally created for, we should
still permit setting a custom path for the python virtual environment.

Closes espressif#15006

Signed-off-by: Frantisek Hrbata <[email protected]>
suda-morris and others added 20 commits January 10, 2025 10:14
fix(uart): allow same pin for tx and rx in uart_set_pin; UART_SELECT_READ_NOTIF race conditon fix (v5.4)

See merge request espressif/esp-idf!36250
Added the checks if the URI is empty for the funtions httpd_req_get_url_query_len
and httpd_req_get_url_query_str in httpd_parser.c
ci: target-test job skip installing toolchain, only install python env (v5.4)

See merge request espressif/esp-idf!34905
…5.4'

fixed interrupt WDT when shutdown bt controller on ESP32(ba6739f) (v5.4)

See merge request espressif/esp-idf!36176
test(system): mark gdb test runners properly (v5.4)

See merge request espressif/esp-idf!35586
feat(tools): Enforced utf-8 encoding with Python open() functions (v5.4)

See merge request espressif/esp-idf!36125
…v5.4'

ci(i2s): re-enable example pytest exceptions (v5.4)

See merge request espressif/esp-idf!36312
…v5.4'

feat(802.15.4): support register isr callback (v5.4)

See merge request espressif/esp-idf!36198
fix(nimble): Fix few nimble issues 11012025 (v5.4)

See merge request espressif/esp-idf!36310
fix(gdma): stop the axi gdma gracefully on CPU SW reset (v5.4)

See merge request espressif/esp-idf!36283
….4' into 'release/v5.4'

feat(https_server): Added checks to verify if uri is empty (v5.4)

See merge request espressif/esp-idf!36286
@CLAassistant
Copy link

CLAassistant commented Jan 15, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 22 committers have signed the CLA.

✅ AlexBethel
✅ ESPAbhinav
✅ hfudev
❌ ESP-Marius
❌ rahult-github
❌ chshu
❌ kumekay
❌ songruo
❌ weiyuhannnn
❌ Isl2017
❌ suda-morris
❌ Jiang Jiang Jian
❌ dobairoland
❌ erhankur
❌ esp-cjh
❌ espressif-bot
❌ zwx1995esp
❌ Astha-cpu
❌ hrushikesh430
❌ mahavirj
❌ Gui71
❌ L-KAYA


Jiang Jiang Jian seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

Warnings
⚠️

Some issues found for the commit messages in this PR:

  • the commit message "Atualização dos componentes de Bluetooth, Wi-Fi e outras dependências":
    • summary looks empty
    • type/action looks empty
  • the commit message "Merge branch 'fix/coredump-gcc-analyzer-warnings_v5.4' into 'release/v5.4'":
    • probably contains Jira ticket reference (GCC-14). Please remove Jira tickets from commit messages.
  • the commit message "Update slave.c to fix the mode":
    • body's lines must not be longer than 100 characters
    • summary looks empty
    • type/action looks empty
  • the commit message "fix(ble): fixed that the resolvable private address does not change when light sleep is enabled":
    • summary appears to be too long

Please fix these commit messages - here are some basic tips:

  • follow Conventional Commits style
  • correct format of commit message should be: <type/action>(<scope/component>): <summary>, for example fix(esp32): Fixed startup timeout issue
  • allowed types are: change,ci,docs,feat,fix,refactor,remove,revert,test
  • sufficiently descriptive message summary should be between 20 to 72 characters and start with upper case letter
  • avoid Jira references in commit messages (unavailable/irrelevant for our customers)

TIP: Install pre-commit hooks and run this check when committing (uses the Conventional Precommit Linter).

⚠️ Please consider squashing your 250 commits (simplifying branch history).
Messages
📖 This PR seems to be quite large (total lines of code: 349883), you might consider splitting it into smaller PRs

👋 Hello Gui71, we appreciate your contribution to this project!


📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more.

🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project.

Click to see more instructions ...


This automated output is generated by the PR linter DangerJS, which checks if your Pull Request meets the project's requirements and helps you fix potential issues.

DangerJS is triggered with each push event to a Pull Request and modify the contents of this comment.

Please consider the following:
- Danger mainly focuses on the PR structure and formatting and can't understand the meaning behind your code or changes.
- Danger is not a substitute for human code reviews; it's still important to request a code review from your colleagues.
- Resolve all warnings (⚠️ ) before requesting a review from human reviewers - they will appreciate it.
- Addressing info messages (📖) is strongly recommended; they're less critical but valuable.
- To manually retry these Danger checks, please navigate to the Actions tab and re-run last Danger workflow.

Review and merge process you can expect ...


We do welcome contributions in the form of bug reports, feature requests and pull requests via this public GitHub repository.

This GitHub project is public mirror of our internal git repository

1. An internal issue has been created for the PR, we assign it to the relevant engineer.
2. They review the PR and either approve it or ask you for changes or clarifications.
3. Once the GitHub PR is approved, we synchronize it into our internal git repository.
4. In the internal git repository we do the final review, collect approvals from core owners and make sure all the automated tests are passing.
- At this point we may do some adjustments to the proposed change, or extend it by adding tests or documentation.
5. If the change is approved and passes the tests it is merged into the default branch.
5. On next sync from the internal git repository merged change will appear in this public GitHub repository.

Generated by 🚫 dangerJS against dfc5cc4

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jan 15, 2025
@github-actions github-actions bot changed the title V5.4 V5.4 (IDFGH-14430) Jan 15, 2025
@Alvin1Zhang
Copy link
Collaborator

Thanks for contribution, what are you planning to fix? Thanks.

@Alvin1Zhang
Copy link
Collaborator

Thanks for reporting, feel free to reopen.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Won't Do This will not be worked on and removed Status: Opened Issue is new labels Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Won't Do This will not be worked on Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.