Skip to content

fix(spotify): include sid/sn in track enqueue uri#6

Open
WinnCook wants to merge 1 commit intosteipete:mainfrom
WinnCook:codex/spotify-track-sid-sn
Open

fix(spotify): include sid/sn in track enqueue uri#6
WinnCook wants to merge 1 commit intosteipete:mainfrom
WinnCook:codex/spotify-track-sid-sn

Conversation

@WinnCook
Copy link
Copy Markdown

@WinnCook WinnCook commented Mar 7, 2026

Fixes #5

Repro

sonos play spotify "August 10 Khruangbin" --name "Kitchen"

What changed

  • try x-sonos-spotify:<encoded-id>?sid=<service>&sn=0 first for track/episode queue URIs
  • preserve existing DIDL metadata and service-token descriptor generation
  • keep the two legacy URI forms as explicit fallbacks for compatibility

Validation

  • added a regression fixture where AddURIToQueue succeeds only when the sid/sn form is used
  • added a fallback fixture where the sid/sn form returns UPnP 800 and the legacy URI still succeeds
  • go test ./internal/sonos ./internal/cli

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UPnP error 800 on AddURIToQueue for Spotify tracks

1 participant