search

feelfreelinux / cspot

A Spotify Connect player targeting, but not limited to embedded devices (ESP32).
Other
495 stars 46 forks source link

Implement partial responses support in MercuryManager #171

Closed tobiasguyer closed 4 months ago

tobiasguyer commented 4 months ago

The mercury manager failed to merge truncated gzip messages. I rewrote the response decoding to be compüatible with trucated messages. I changed the structure of the response handling slightly, so it would be able to handle incoming asynchronous messages.

feelfreelinux commented 4 months ago

Thanks for your contribution, will try to test it out soon :) What kind of messages from spotify are sent in gzip form - probably missed out on it during the implementation

feelfreelinux commented 4 months ago

Took the liberty of doing some minor fixes :) Merging into the develop branch, thanks a lot for your contribution!

tobiasguyer commented 4 months ago

thanks a lot for the changes. I'm not exactly sure if those responses are gzipped, but at least they share the structure and there are no more problems with long playlists :)