search

koel / docker

A minimal docker image for the koel music streaming server.
https://hub.docker.com/r/phanan/koel/
MIT License
178 stars 54 forks source link

Fix locale #91

Closed glynnt closed 3 years ago

glynnt commented 3 years ago

For song which needs ffmpeg to transcode and its path contains chinese characters, it will fail because the characters removed from the path passed to ffmpeg. This fix is to change the default locale to en_US.UTF-8 which fixes this issue. (Other UTF8 locale should work)

Hyzual commented 3 years ago

Hi, thanks for the PR 🙂 ! I'll take a look when I can, I can't promise a time but I'll test it this week

Hyzual commented 3 years ago

Hi, sorry for the (very) long delay. I managed to reproduce and test your fix. It was a long-standing issue, I had always thought it had something to do with koel itself, I'm glad it's resolved. Thanks a lot :) I squashed your commits and grouped the instructions to avoid creating too many layers.