htilly / zenmusic

Slack / Sonos / Spotify / Node.js - Control Sonos through #Slack
125 stars 36 forks source link

Merging with changes from vibol #6

Closed htilly closed 8 years ago

htilly commented 8 years ago

Thanks for helping out!

vibol commented 8 years ago

Thanks for the great tool!

htilly commented 8 years ago

Quick question. The error 500 was solved by replacing: sonos.addSpotifyQueue(spid, function (err, res) { with sonos.addSpotify(spid, function (err, res) { Right? Nothing else?

There are others receiving error 500 as well. Strangely everything works fine for us. And if I change to the new function, I start receiving 500 errors ;)

BR Tilly

vibol commented 8 years ago

Yes that's right. Changing that call to addSpotify made it work for us. Perhaps there's a difference in Sonos devices that causes this. I didn't have time to debug the calls in depth but I noticed the addSpotifyQueue call had some additional parameters in the XML payload going out to he Sonos.