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

Navidrome not working if downloads are disabled (transcoding) #201

Closed
stonie08 opened this issue Nov 12, 2023 · 6 comments
Closed

Navidrome not working if downloads are disabled (transcoding) #201

stonie08 opened this issue Nov 12, 2023 · 6 comments
Labels
enhancement New feature or request implemented This feature/bug is implemented TestInBeta This feature can be tested in Beta

Comments

@stonie08
Copy link

I can't play any music with Amperfy 1.1.3 and Navidrome 0.49.3 if downloads are disabled on the server side. If "Auto cache played songs" is enabled in the app I get "API Error Status Code: 50 downloads are disabled", which seems fine given that caching the songs kinda implies downloading them. However, if the auto cache function is disabled, I don't get an error message but no music is playing either. Other clients can play music fine with the same Navidrome instance and config.

Steps to reproduce:

  • Run Navidrome 0.49.3
  • in the Navidrome config set EnableDownloads = false
  • Try to play music in Amperfy
@BLeeEZ
Copy link
Owner

BLeeEZ commented Apr 16, 2024

I think this error could be fixed when Amperfy gets transcoding support in the next big version.

@BLeeEZ BLeeEZ changed the title Navidrome not working if downloads are disabled Navidrome not working if downloads are disabled (transcoding) Apr 22, 2024
@BLeeEZ BLeeEZ added the enhancement New feature or request label Apr 22, 2024
@BLeeEZ
Copy link
Owner

BLeeEZ commented Apr 22, 2024

Transcoding is implemented in 16e9bed.

@BLeeEZ BLeeEZ added the implemented This feature/bug is implemented label Apr 22, 2024
@BLeeEZ BLeeEZ added the TestInBeta This feature can be tested in Beta label Jun 5, 2024
@BLeeEZ
Copy link
Owner

BLeeEZ commented Jun 5, 2024

This feature can be tested in the newest beta v1.2.1 (2): #25

@BLeeEZ
Copy link
Owner

BLeeEZ commented Jun 15, 2024

Did you have time test the feature? Is it behaving as you expected?

@stonie08
Copy link
Author

I just tried it and played around with the settings a bit. Overall it seems to work fine now. I tried some configurations for transcoding and caching and couldn't make the app not play a song. Thank you!

A small UX thing I did notice is that if downloads are disabled on the server and in the app I set a Max bitrate for streaming (wifi and cellular), set to streaming format (transcoding) to Raw/Original and the Cache Format (Transcoding) to Raw/ Original I get a big red "Download Status Code: 50 downloads are disabled", which makes sense given the descirpiton below on the settings page, but is a bit intrusive given that the music plays fine and the message reappears for every single song.

Likely worth mentioning, but also separate issue: the description below "Cache Format (Transcoding)" mentions a stream setting for server side transcoding, but there's no "stream" setting.

@BLeeEZ
Copy link
Owner

BLeeEZ commented Jun 16, 2024

To avoid this error message you can do two things: Either activate transcoding for cache or deactivate "Auto cache played Songs".

The mentioned description: If you activate cache transcoding, Amperfy needs to use the Subsonic API action "stream" since the Subsonic API action "download" does not allow to provide a transcoding format. I display that info so the user knows that even if download is not allowed on server side it is still possible to save songs for offline play. I heard from other users that they don't want to use the API action "stream" and always want to use the API action "download".

@BLeeEZ BLeeEZ closed this as completed Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request implemented This feature/bug is implemented TestInBeta This feature can be tested in Beta
Projects
None yet
Development

No branches or pull requests

2 participants