[GH-ISSUE #1179] Support ffmpeg audio filters #676

New issue

Closed

opened 2026-02-26 02:33:56 +03:00 by kerem · 1 comment

kerem commented

2026-02-26 02:33:56 +03:00

Owner

Originally created by @JD557 on GitHub (Feb 8, 2020).
Original GitHub issue: https://github.com/koel/koel/issues/1179

Description
Since there's already a transcoding streamer that uses ffmpeg (https://github.com/phanan/koel/blob/master/app/Services/Streamers/TranscodingStreamer.php).

I think it should be pretty simple and helpful to add some optional ffmpeg filters, such as:

(I haven't tested those flags, so I'm not sure if they are suitable for streaming)

I think that (at least for a first version), it would be enough to add this as a config/koel.php.

I would send a PR for this, but I'm not sure if the TranscodingStreamer is always used (I'm pretty sure its not, since I had koel working for a while with a misconfigured ffmpeg).

This feature would have to force the TranscodingStreamer to always be used, and I'm not familiar with the codebase to know where those checks are performed.

Originally created by @JD557 on GitHub (Feb 8, 2020). Original GitHub issue: https://github.com/koel/koel/issues/1179 **Description** Since there's already a transcoding streamer that uses ffmpeg (https://github.com/phanan/koel/blob/master/app/Services/Streamers/TranscodingStreamer.php). I think it should be pretty simple and helpful to add some optional ffmpeg filters, such as: - [loudnorm](http://ffmpeg.org/ffmpeg-all.html#loudnorm) - [dynaudnorm](http://ffmpeg.org/ffmpeg-all.html#dynaudnorm). - [earwax](http://ffmpeg.org/ffmpeg-all.html#earwax) (I haven't tested those flags, so I'm not sure if they are suitable for streaming) I think that (at least for a first version), it would be enough to add this as a `config/koel.php`. I would send a PR for this, but I'm not sure if the `TranscodingStreamer` is always used (I'm pretty sure its not, since I had koel working for a while with a misconfigured ffmpeg). This feature would have to force the `TranscodingStreamer` to always be used, and I'm not familiar with the codebase to know where those checks are performed.

kerem closed this issue

2026-02-26 02:33:56 +03:00

kerem commented

2026-02-26 02:33:56 +03:00

Author

Owner

@phanan commented on GitHub (Feb 11, 2020):

Thanks for the idea. TranscodingStreamer is and should be only used for audios that require transcoding. While filters can be good for some, I doubt they are the norms and should determine the usage of the streamer. Not to mention, enforcing TranscodingStreamer, at least for now, would break some existing functionalities e.g. fast-forwarding and client caching.

@phanan commented on GitHub (Feb 11, 2020): Thanks for the idea. `TranscodingStreamer` is and should be only used for audios that require transcoding. While filters can be good for some, I doubt they are the norms and should determine the usage of the streamer. Not to mention, enforcing `TranscodingStreamer`, at least for now, would break some existing functionalities e.g. fast-forwarding and client caching.

kerem referenced this issue

2026-02-26 03:30:34 +03:00

[PR #676] [MERGED] Updated to Mockery 1 #1318

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/koel-koel#676

No description provided.

Rows
Columns