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

Test Server function picks too large playlist #97

Open
Clusters opened this issue Jul 26, 2024 · 1 comment
Open

Test Server function picks too large playlist #97

Clusters opened this issue Jul 26, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@Clusters
Copy link

Clusters commented Jul 26, 2024

I'm currently struggling to get my Navidrome (Subsonic) server running with this amazing app!

One of my issues was, that I received a HTTP Error 402 response too large error when using the Test Server feature.
After some debugging I stumbled over the SubMusicTestView class. It seems that it uses the first playlist with songs in it. If it happens that the first playlist has too many songs in it, it will throw this error.
In my case (Navidrome) the first JSON entry from the playlist fetch response had 100 songs in it. As Navidrome sorts the items ascending by name, I just had to create a new playlist with a name ordered above that playlist and add a single song to it. This might be a workaround for someone out there.

However, this seemed just to be one brick in my solution to get the app running. Might open a separate issue for that.

@memen45
Copy link
Owner

memen45 commented Jul 26, 2024

This correct. Based purely on the number of songs, one cannot determine whether the playlist will be too large for memory or not. There are two factors to this:

  • the available memory is dynamic and cannot be known.
  • the response size is dependent on the amount of metadata that is present for each of the songs.

A possible solution could be to just try the next playlist upon receiving a Response too large error. This is still no guarantee, as all playlists may be too large.

Also, I should note this is purely a Subsonic API issue, as all other APIs support paging for the responses.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants