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

[Bug] ALAC files don't stream directly (iOS 0.6.1) #225

Closed
TinDefacto opened this issue Apr 28, 2022 · 7 comments
Closed

[Bug] ALAC files don't stream directly (iOS 0.6.1) #225

TinDefacto opened this issue Apr 28, 2022 · 7 comments
Milestone

Comments

@TinDefacto
Copy link
Contributor

Hi there!

First off, LOVE the app, and it's made listening to music such a better experience when away from home. Thank you!!

I just found out the app had been updated and was elated to find so many of the changes I'd been looking for. Alas, for some reason, the update seems to have caused a new issue: any ALAC files on my Jellyfin server will not play directly. They'll work if transcoded, and any file that isn't ALAC works perfectly fine without transcoding. But since 97% of my library is in ALAC, and I'd prefer to stream directly, this is a bit of an issue!

My apologies if this has been covered elsewhere. I'm not sure what could have changed about the update to disturb this playback, but I hope it's not too difficult to fix!

Thanks again!

@jmshrv
Copy link
Owner

jmshrv commented Apr 29, 2022

This may be an issue with the audio plugin I'm using, I'll see if I can find where ALAC broke since I'm pretty sure it worked in the last release

@jmshrv jmshrv added this to the 0.6.2 milestone Apr 29, 2022
@TinDefacto
Copy link
Contributor Author

Yes, ALAC definitely worked fine before I updated! (Though I did skip 0.6.0.)

@jmshrv
Copy link
Owner

jmshrv commented May 12, 2022

Found the bug, it's here: https://github.com/UnicornsOnLSD/finamp/blob/master/lib/services/MusicPlayerBackgroundTask.dart#L541

Safari includes m4a|alac in its Container parameter, I'll clean this up to properly select the codec for each platform.

@jmshrv
Copy link
Owner

jmshrv commented May 18, 2022

Sort of fixed in 40d9a6d by adding m4a|alac to the supported containers, I'll fix this up properly at some point (i.e. actually checking the supported codecs on-device)

@csaavedra
Copy link

Sorry to relive this, but I noticed today that ALAC is not playing when streaming. I have the latest Android release from the Google Play store. Is the bug fix above already released?

@jmshrv
Copy link
Owner

jmshrv commented Dec 5, 2022

ALAC isn't supported in ExoPlayer, so it can't work in the current system. When I get around to #177, it'll work, but that's ages away. If I were you I'd consider converting your music to FLAC if possible, but I understand that doing that is really inconvenient (especially if you value the idea of keeping the exact files over the decades).

@csaavedra
Copy link

No worries. It's a lossless format so converting to FLAC shouldn't change anything.

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

No branches or pull requests

3 participants