[PR #1944] [MERGED] feat: support transcoding for non-local songs #1968

Closed
opened 2026-02-26 03:32:54 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/1944
Author: @phanan
Created: 6/7/2025
Status: Merged
Merged: 6/7/2025
Merged by: @phanan

Base: masterHead: transcode-remote


📝 Commits (1)

  • 98ea326 feat: supports transcoding for non-local songs

📊 Changes

18 files changed (+366 additions, -172 deletions)

View changed files

📝 app/Http/Resources/SongResource.php (+1 -1)
📝 app/Services/SongStorages/CloudStorage.php (+11 -0)
📝 app/Services/SongStorages/DropboxStorage.php (+1 -1)
📝 app/Services/SongStorages/LocalStorage.php (+1 -1)
📝 app/Services/SongStorages/S3LambdaStorage.php (+1 -1)
📝 app/Services/SongStorages/SftpStorage.php (+1 -1)
📝 app/Services/Streamer/Adapters/TranscodingStreamerAdapter.php (+12 -3)
📝 app/Services/Streamer/Streamer.php (+2 -41)
app/Services/Transcoder.php (+95 -0)
📝 app/Values/TranscodeResult.php (+2 -41)
📝 config/koel.php (+2 -2)
📝 docs/usage/streaming.md (+2 -12)
📝 tests/Feature/SongPlayTest.php (+2 -2)
📝 tests/Integration/Services/Streamer/StreamerTest.php (+8 -9)
tests/Integration/Services/TranscoderTest.php (+165 -0)
tests/Integration/Values/TranscodeResultTest.php (+0 -57)
📝 tests/TestCase.php (+11 -0)
tests/Unit/Services/SongStorages/CloudStorageTest.php (+49 -0)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/koel/koel/pull/1944 **Author:** [@phanan](https://github.com/phanan) **Created:** 6/7/2025 **Status:** ✅ Merged **Merged:** 6/7/2025 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `transcode-remote` --- ### 📝 Commits (1) - [`98ea326`](https://github.com/koel/koel/commit/98ea3265174a62e8a9cdbb081492fd6285ad16eb) feat: supports transcoding for non-local songs ### 📊 Changes **18 files changed** (+366 additions, -172 deletions) <details> <summary>View changed files</summary> 📝 `app/Http/Resources/SongResource.php` (+1 -1) 📝 `app/Services/SongStorages/CloudStorage.php` (+11 -0) 📝 `app/Services/SongStorages/DropboxStorage.php` (+1 -1) 📝 `app/Services/SongStorages/LocalStorage.php` (+1 -1) 📝 `app/Services/SongStorages/S3LambdaStorage.php` (+1 -1) 📝 `app/Services/SongStorages/SftpStorage.php` (+1 -1) 📝 `app/Services/Streamer/Adapters/TranscodingStreamerAdapter.php` (+12 -3) 📝 `app/Services/Streamer/Streamer.php` (+2 -41) ➕ `app/Services/Transcoder.php` (+95 -0) 📝 `app/Values/TranscodeResult.php` (+2 -41) 📝 `config/koel.php` (+2 -2) 📝 `docs/usage/streaming.md` (+2 -12) 📝 `tests/Feature/SongPlayTest.php` (+2 -2) 📝 `tests/Integration/Services/Streamer/StreamerTest.php` (+8 -9) ➕ `tests/Integration/Services/TranscoderTest.php` (+165 -0) ➖ `tests/Integration/Values/TranscodeResultTest.php` (+0 -57) 📝 `tests/TestCase.php` (+11 -0) ➕ `tests/Unit/Services/SongStorages/CloudStorageTest.php` (+49 -0) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 03:32:54 +03:00
Sign in to join this conversation.
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#1968
No description provided.