Skip to content

feat: add support for overriding default multicast interface#208

Merged
svrooij merged 1 commit intosvrooij:betafrom
gcbartlett:sonos-cli-22
Nov 19, 2025
Merged

feat: add support for overriding default multicast interface#208
svrooij merged 1 commit intosvrooij:betafrom
gcbartlett:sonos-cli-22

Conversation

@gcbartlett
Copy link

  • fix: sonos-cli-22

Add support for overriding default multicast interface.

Description

This pull request adds the ability to override the default multicast interface used to send SSDP discovery requests.

As described in sonos-cli #22, if the user has activated a WSL environment, or has installed the Hyper-V subsystem, their default multicast interface will change to one of the virtual vEthernet interfaces added by WSL/Hyper-V, instead of their physical network interface. This causes a (sonos-cli) sonos zones request to fail and timeout without discovering any SONOS devices (the query is sent out on the virtual interface).

To work around this issue, with this change, you can set either the SONOS_MULTICAST_IF or IP_MULTICAST_IF environment variable to the IP address of the network interface you want to be used to discover the SONOS devices.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 19285653215

Details

  • 3 of 5 (60.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 70.907%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/sonos-device-discovery.ts 3 5 60.0%
Totals Coverage Status
Change from base Build 13506646266: -0.02%
Covered Lines: 1526
Relevant Lines: 2125

💛 - Coveralls

@svrooij svrooij merged commit 9b34aa0 into svrooij:beta Nov 19, 2025
9 checks passed
@github-actions
Copy link

🎉 This PR is included in version 2.6.0-beta.13 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants