homebridge / ciao

RFC 6762 and RFC 6763 compliant mdns service discovery library written in Typescript
MIT License
79 stars 6 forks source link

Add support for IPv4-mapped IPv6 addresses #43

Closed donavanbecker closed 1 week ago

donavanbecker commented 2 weeks ago

:recycle: Current situation

Currently IPv4-mapped IPv6 addresses are not supported

:bulb: Proposed solution

Add support for IPv4-mapped IPv6 addresses

:gear: Release Notes

Added support for IPv4-mapped IPv6 addresses

:heavy_plus_sign: Additional Information

Not sure if there is anything else that needs to be changed?

Testing

Install tvOS 18 or macOS 15 beta

or

a device that has an IPv4-mapped IPv6 address.

Reviewer Nudging

tvOS 18 is what is causing the issue indicated in #41, but others have reported it with macOS 15 beta.

bwp91 commented 2 weeks ago

i’m afraid i won’t be a good reviewer of this as (1) i don’t enough about this subject and best practises etc and (2) i cant even reproduce locally to check whether the changes is general work or not!

donavanbecker commented 2 weeks ago

@hjdhjd, Homebridge v1.8.4-alpha.18 has these changes and looks like it is working as expected. I will remove my commented out sections.

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9816393432

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/util/v4mapped.ts 6 7 85.71%
src/util/domain-formatter.ts 15 22 68.18%
<!-- Total: 30 38 78.95% -->
Totals Coverage Status
Change from base Build 9729372371: -0.2%
Covered Lines: 1190
Relevant Lines: 3084

💛 - Coveralls