mirror of
https://github.com/librespot-org/librespot.git
synced 2026-04-27 08:15:50 +03:00
[GH-ISSUE #223] Crash on connect #150
Labels
No labels
A-Alsa
SpotifyAPI
Tokio 1.0
audio
bug
can't reproduce
compilation
dependencies
duplicate
enhancement
good first issue
help wanted
high priority
imported
imported
invalid
new api
pull-request
question
reverse engineering
wiki
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/librespot#150
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @nzoschke on GitHub (May 17, 2018).
Original GitHub issue: https://github.com/librespot-org/librespot/issues/223
On the latest project version on OS X, I see the device in Spotify Connect but librespot crashes when I connect to it:
@dawidd6 commented on GitHub (May 17, 2018):
Yup, can confirm that. Same with Android and Ubuntu clients.
@omelettedufromagee commented on GitHub (May 17, 2018):
I can confirm too. Crashed on me while listening to music around 11.30 EST. Spotify probably updated something.
I am using the librespot version built on 2018-04-30.
@analogic commented on GitHub (May 17, 2018):
Same here https://github.com/dtcooper/raspotify/issues/113
@plietar commented on GitHub (May 17, 2018):
Can you try to add a new entry in
https://github.com/librespot-org/librespot/blob/master/protocol/proto/spirc.proto#L78, with value0xe, see if that fixes it?You'll have to rebuild the sources using the
protocol/build.shfile.@nzoschke commented on GitHub (May 17, 2018):
I am trying but having trouble with the build environment:
@nzoschke commented on GitHub (May 17, 2018):
Ok I got past that with
cargo install protobuf --version 1.2.1.Adding the
0xevalue does fix it!@plietar commented on GitHub (May 17, 2018):
Great! Could you make a PR with the updated
.protoand sources please.The good news is rust-protobuf has recently been fixed to ignore unknown values: https://github.com/stepancheg/rust-protobuf/pull/276
This was released in version 1.5.0, so if we update to that version we shouldn't run into this issue.
I'll send a PR soon for this, and also use protobuf-codegen-pure so we don't have to manually regenerate sources.
@nzoschke commented on GitHub (May 17, 2018):
I’m happy to however I stepped away for my dev machine for the day so it’ll
be later tonight.
I am not sure what to name the field either...
On Thu, May 17, 2018 at 10:11 AM Paul Liétar notifications@github.com
wrote:
@dawidd6 commented on GitHub (May 17, 2018):
Ok it's working now. Needed to add this line:
kSupportsPlaylistV3 = 0xeunder this line:
https://github.com/librespot-org/librespot/blob/master/protocol/proto/spirc.proto#L78
I made a pull request for this. @nzoschke if you don't mind?
@nzoschke commented on GitHub (May 17, 2018):
@dawidd6 don't mind at all. Thanks!
@berrywhite96 commented on GitHub (May 17, 2018):
Compiled on my mac, same error like before :/
@dawidd6 commented on GitHub (May 17, 2018):
@berrywhite96 did you compile from my fork? I just made fresh build from my master and it's working as expected.
@michalfita commented on GitHub (May 17, 2018):
I have the same problem on Raspberry. When we can expect a fix on master?
@Gronis commented on GitHub (May 17, 2018):
I've complied from @nzoschke commit on linux x86 and it works fine :)
@michalfita We will get a fix on master once the PR is merged
@berrywhite96 commented on GitHub (May 17, 2018):
@dawidd6 oh I thought its already in the master branch :D. Your fork work as expected on mac! Will now compile on raspberry, thank you.
@dtcooper commented on GitHub (May 17, 2018):
Tracking this for Raspotify as well!
@plietar commented on GitHub (May 17, 2018):
Btw what other devices with what version of the Spotify app do you have on when this issue happens?
I'm curious as to what the meaning of the new enum is, but neither the macOS desktop client nor the Android one support it yet (at least as far as I can tell, from a quick dump of strings).
@dawidd6 commented on GitHub (May 17, 2018):
Android (phone) - 8.4.36.305
Ubuntu (desktop) - 1.0.77.338.g758ebd78
@sashahilton00 commented on GitHub (May 17, 2018):
@plietar it could be the Time Capsule playlist, for example. I get a funky new UI for Spotify generated playlists now. Though that could also be the v2 playlist
@dtcooper commented on GitHub (May 17, 2018):
Confirming that @dawidd6's fork works on Raspberry Pi and closes Raspotify:#114.
@berrywhite96 commented on GitHub (May 17, 2018):
@plietar Mac OS: 1.0.80.474.gef6b503e
Android: 8.4.53.703
Both dont connect.
cargo --lockedeverywhere #812