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

GetZoneGroupTopology fails with http status 500. #131

Closed
ebaauw opened this issue Apr 12, 2020 · 3 comments
Closed

GetZoneGroupTopology fails with http status 500. #131

ebaauw opened this issue Apr 12, 2020 · 3 comments
Labels

Comments

@ebaauw
Copy link
Owner

ebaauw commented Apr 12, 2020

From Discord: zone player responds with status 500 on POST to /ZoneGroupTopology/Control for GetZoneGroupTopology:

2020-04-12T19:15:20.993Z ZpClient:request:1 post("ZonePlayer", "ZoneGroupTopology", "GetZoneGroupState", {})
2020-04-12T19:15:20.993Z HttpClient1 request(POST, /ZoneGroupTopology/Control, "<s:Envelope xmlns:s=\"http://schemas.xmlsoap.org/soap/envelope/\" s:encodingStyle=\"http://schemas.xmlsoap.org/soap/encoding/\"><s:Body><u:GetZoneGroupState xmlns:u=\"urn:schemas-upnp-org:service:ZoneGroupTopology:1\"></u:GetZoneGroupState></s:Body></s:Envelope>", {"SOAPaction":"\"urn:schemas-upnp-org:service:ZoneGroupTopology:1#GetZoneGroupState\"","content-type":"text/xml; charset=utf-8"})
zp: error: POST /ZoneGroupTopology/Control: http status 500 Internal Server Error

It's a network of 30 zone players - might be stretching it (household can hold a max of 32 zone players). The Sonos app doesn't make this call, but subscribes to ZoneGroupTopology events of the associated zone player.

Unfortunately, ZpClient makes this call when connecting to the zone player, so need to find a way around it. The ZoneGroupTopology events look good, though, see attached output of zp eventlog: zpeventlog.zip

@ebaauw ebaauw added the bug label Apr 12, 2020
ebaauw added a commit that referenced this issue Apr 14, 2020
Dirty hack to workaround #131.
@ebaauw
Copy link
Owner Author

ebaauw commented Jul 18, 2021

In beta v1.4.0-1, zp topology -v uses the topology from the connected zone player to augment the device description of the other zone players. Still need to apply that logic to Homebridge ZP.

Also, the household ID is only reported in the ZoneGroupTopoloy event, but not in the GetZoneGroupTopology response.

@ebaauw
Copy link
Owner Author

ebaauw commented Jul 27, 2021

As of beta v1.4.0-4, Homebridge ZP no longer tries the GetZoneGroupTopology and uses events straight away.

@ebaauw
Copy link
Owner Author

ebaauw commented Jul 31, 2021

In v1.4.0.

@ebaauw ebaauw closed this as completed Jul 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant