Make udp component able to send and listen to IPv6 multicast
What does this implement/fix?
Types of changes
- [ ] Bugfix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Code quality improvements to existing code or addition of tests
- [ ] Other
Related issue or feature (if applicable):
- fixes
Pull request in esphome-docs with documentation (if applicable):
- esphome/esphome-docs#4891
Test Environment
- [x] ESP32
- [x] ESP32 IDF
- [ ] ESP8266
- [ ] RP2040
- [ ] BK72xx
- [ ] RTL87xx
Example entry for config.yaml:
# Example config.yaml
Checklist:
- [x] The code change is tested and works locally.
- [ ] Tests have been added to verify that the new code works (under
tests/folder).
If user exposed functionality or configuration variables are added/changed:
- [ ] Documentation added/updated in esphome-docs.
Hey there @clydebarrow, mind taking a look at this pull request as it has been labeled with an integration (udp) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)
Codecov Report
:x: Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 72.45%. Comparing base (fd64585) to head (3bdd7c6).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| esphome/config_validation.py | 16.66% | 5 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## dev #8533 +/- ##
==========================================
- Coverage 72.45% 72.45% -0.01%
==========================================
Files 53 53
Lines 11114 11117 +3
Branches 1501 1501
==========================================
+ Hits 8053 8055 +2
- Misses 2672 2673 +1
Partials 389 389
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks :+1:
To use the changes in this PR:
# Clone the repository:
git clone https://github.com/esphome/esphome
cd esphome
# Checkout the PR branch:
git fetch origin pull/8533/head:udp-ipv6
git checkout udp-ipv6
# Install the development version:
script/setup
# Activate the development version:
source venv/bin/activate
Now you can run esphome as usual to test the changes in this PR.
(Added by the PR bot)
👋 Hi there! This PR modifies 5 file(s) with codeowners.
@esphome/core - As codeowner(s) of the affected files, your review would be appreciated! 🙏
Note: Automatic review request may have failed, but you're still welcome to review.
Does this also add unicast support? Now that ESPHome includes a syslog component which is based on the udp component, it's a regression from the external one which I was previously using that supported IPv6.
Does this also add unicast support? Now that ESPHome includes a syslog component which is based on the udp component, it's a regression from the external one which I was previously using that supported IPv6.
Yes
Does this udp ipv6 also work for openthread? specially with package transport?
Any progress on this? I'd like to fix the syslog again; it regressed into the 1990s when it was merged. The external component used to support IPv6.
@dwmw2 I bumped into a problem with libretiny-devices. I think the Arduino code needs to be upgraded.
What's the LibreTiny issue? I already upgraded the LWIP there last year, I think.