esphome icon indicating copy to clipboard operation
esphome copied to clipboard

Make udp component able to send and listen to IPv6 multicast

Open HeMan opened this issue 1 year ago • 3 comments

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:

HeMan avatar Apr 08 '25 14:04 HeMan

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)

probot-esphome[bot] avatar Apr 08 '25 14:04 probot-esphome[bot]

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.

codecov-commenter avatar Apr 08 '25 14:04 codecov-commenter

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks :+1:

Learn more about our pull request process.

esphome[bot] avatar Apr 09 '25 13:04 esphome[bot]

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)

github-actions[bot] avatar Jul 21 '25 05:07 github-actions[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.

github-actions[bot] avatar Jul 21 '25 05:07 github-actions[bot]

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.

dwmw2 avatar Jul 22 '25 10:07 dwmw2

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

HeMan avatar Aug 06 '25 05:08 HeMan

Does this udp ipv6 also work for openthread? specially with package transport?

DJJo14 avatar Oct 15 '25 07:10 DJJo14

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 avatar Nov 19 '25 14:11 dwmw2

@dwmw2 I bumped into a problem with libretiny-devices. I think the Arduino code needs to be upgraded.

HeMan avatar Nov 19 '25 18:11 HeMan

What's the LibreTiny issue? I already upgraded the LWIP there last year, I think.

dwmw2 avatar Nov 19 '25 18:11 dwmw2