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

Fix: Deezer tracks that have been replaced with a newer version fail to play #1892

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

awhiemstra
Copy link
Contributor

Some tracks have been replaced by a newer version. When this happens, the old track information stays available but it will fail to play because the stream has become invalid. Fortunately, the playback API provides the track replacement in a "FALLBACK" entry. So when we encounter a track that has a "FALLBACK" entry, use the information from that entry rather than the main track entry for the stream.

Fixes music-assistant/support#3466

Copy link
Member

@marcelveldt marcelveldt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work in finding this out @awhiemstra !

@marcelveldt
Copy link
Member

ah the linter found a small issue, can you correct please ?

…to play

Some tracks have been replaced by a newer version. When this happens,
the old track information stays available but it will fail to play
because the stream has become invalid. Fortunately, the playback API
provides the track replacement in a "FALLBACK" entry. So when we
encounter a track that has a "FALLBACK" entry, use the information from
that entry rather than the main track entry for the stream.

Fixes music-assistant/support#3466
@awhiemstra
Copy link
Contributor Author

Should be fixed now.

@marcelveldt marcelveldt merged commit 933399f into music-assistant:dev Jan 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants