mirror of
https://github.com/koel/koel.git
synced 2026-04-26 09:15:59 +03:00
[GH-ISSUE #1356] Error 500 when downloading 1 song #782
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#782
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 @masterT on GitHub (Sep 22, 2021).
Original GitHub issue: https://github.com/koel/koel/issues/1356
Describe the bug
The server returns a 500 error when downloading a single song.
To reproduce
Steps to reproduce the behavior:
Expected behavior
Download the song.
Screenshots
I set
APP_DEBUG=trueand got the response:Environment
Docker image hyzual/koel with docker-compose.
Additional context
I verified and the requested song exists is in the database.
@nielsdrost7 commented on GitHub (Sep 26, 2021):
I've looked in the sourcecode and it cannot determine the type of file that its downloading.
The DownloadService is looking through Songs, Albums, Artists, things like that and it cannot determine the type.
The only way to resolve this is to reproduce it, but there is a chance that with songs (here) it cannot be reproduced.
@masterT commented on GitHub (Sep 28, 2021):
The problem occurred with any songs in my library. I will try to debug this.
@phanan commented on GitHub (Sep 29, 2021):
I can't reproduce this.
@kevincaradant commented on GitHub (Dec 5, 2021):
Hi
Any news on this issue?
I have the same issue.
@masterT, just to know, do you have also an issue when you try to upload a song ?
Thank you
logs:
Docker: docker pull hyzual/koel:latest (today)
PostgresSQL: Postgres 13
@masterT commented on GitHub (Dec 5, 2021):
I used the command line to import my music library and had no issue.
@kevincaradant commented on GitHub (Dec 5, 2021):
OK after many tries I found the issue. It depends of the rights given to the following folders:
-> covers and search-index.
About the import of musics, it's also an issue with rights where the musics is storaged.
@LauraWebdev commented on GitHub (Jan 17, 2022):
I seem to have the same issue, this could also be the reason why the mobile app doesn't seem to play anything.
I've setup the latest docker image and I can confirm that the music files are owned by www-data.
I can listen to the songs from the web UI but not from the mobile app.
Clickiing "Download" on the web UI brings me the known
Unsupported download type. {"userId":1,"exception":"[object] (InvalidArgumentException(code: 0): Unsupported download type.Edit: I seem to be able to download songs on Android. Can't listen to files without downloading them first, though. Still can't download from the web UI.
@nielsdrost7 commented on GitHub (Jan 17, 2022):
@LauraWebdev it cannot determine the type of file that its downloading.
Is there a way you can step through the code, maybe do a dd() in the code?
app/Http/Controllers/Download/SongController.phpCan you do a
dd($songs);here?It leads to DownloadService (app/Services)
public function from($mixed): string(line 31)Can you do a
dd($mixed);there?@LauraWebdev commented on GitHub (Jan 17, 2022):
I'm not sure if I can easily do this in my setup, as I'm running koel in a docker container.
@phanan commented on GitHub (Jan 17, 2022):
This actually looks like a bug in the latest version. Will fix.
@phanan commented on GitHub (Jan 17, 2022):
Tagged a new version: https://github.com/koel/koel/releases/tag/v5.1.13. Thanks everyone!