[PR #6538] [MERGED] Kill ProgressiveFileCopier and seek to end for ongoing livetv #11012

Closed
opened 2026-02-07 06:30:51 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/6538
Author: @cvium
Created: 9/10/2021
Status: Merged
Merged: 9/20/2021
Merged by: @Bond-009

Base: masterHead: livetv_oh_no


📝 Commits (6)

  • 1603d19 Kill ProgressiveFileCopier and seek to end for ongoing livetv
  • f3573b0 Remove the unused arg
  • 1a5a74d Remove more unused args
  • 026a7af Don't throw when livestream file isn't found
  • ff328fe Replace GetDirectStreamProviderByUniqueId with GetLiveStreamInfoByUniqueId
  • ae0055c Missing linebreaks that Rider apparently can't just do for me?

📊 Changes

15 files changed (+157 additions, -393 deletions)

View changed files

📝 Emby.Server.Implementations/Library/ExclusiveLiveStream.cs (+6 -0)
📝 Emby.Server.Implementations/Library/MediaSourceManager.cs (+16 -17)
📝 Emby.Server.Implementations/LiveTv/EmbyTV/DirectRecorder.cs (+12 -4)
📝 Emby.Server.Implementations/LiveTv/TunerHosts/HdHomerun/HdHomerunUdpStream.cs (+1 -6)
📝 Emby.Server.Implementations/LiveTv/TunerHosts/LiveStream.cs (+22 -95)
📝 Emby.Server.Implementations/LiveTv/TunerHosts/SharedHttpStream.cs (+15 -34)
📝 Jellyfin.Api/Controllers/LiveTvController.cs (+3 -3)
📝 Jellyfin.Api/Controllers/VideosController.cs (+8 -12)
📝 Jellyfin.Api/Helpers/AudioHelper.cs (+11 -14)
Jellyfin.Api/Helpers/ProgressiveFileCopier.cs (+0 -187)
📝 Jellyfin.Api/Helpers/ProgressiveFileStream.cs (+24 -11)
📝 Jellyfin.Api/Models/StreamingDtos/StreamState.cs (+3 -0)
MediaBrowser.Controller/Library/IDirectStreamProvider.cs (+19 -0)
📝 MediaBrowser.Controller/Library/ILiveStream.cs (+3 -0)
📝 MediaBrowser.Controller/Library/IMediaSourceManager.cs (+14 -10)

📄 Description

Changes
I was going to just fix the seeking thing but ended up doing a tiny bit more. It's maybe too much at once...

Issues
Fixes #5724


🔄 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/jellyfin/jellyfin/pull/6538 **Author:** [@cvium](https://github.com/cvium) **Created:** 9/10/2021 **Status:** ✅ Merged **Merged:** 9/20/2021 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `livetv_oh_no` --- ### 📝 Commits (6) - [`1603d19`](https://github.com/jellyfin/jellyfin/commit/1603d1928e888abeccc6a5fab2e1f13aa65eea6c) Kill ProgressiveFileCopier and seek to end for ongoing livetv - [`f3573b0`](https://github.com/jellyfin/jellyfin/commit/f3573b061c4d9eb869316ce3de320fd8803aeef8) Remove the unused arg - [`1a5a74d`](https://github.com/jellyfin/jellyfin/commit/1a5a74d2a95506249cf071c659e3c6cf01f28f78) Remove more unused args - [`026a7af`](https://github.com/jellyfin/jellyfin/commit/026a7af0e8cc15f889ba94079c8bc9566a74d505) Don't throw when livestream file isn't found - [`ff328fe`](https://github.com/jellyfin/jellyfin/commit/ff328fefc57461bd999210b3059b997e7f9842ac) Replace GetDirectStreamProviderByUniqueId with GetLiveStreamInfoByUniqueId - [`ae0055c`](https://github.com/jellyfin/jellyfin/commit/ae0055c49483568aa244b4e7fe2f80e24ae2e53a) Missing linebreaks that Rider apparently can't just do for me? ### 📊 Changes **15 files changed** (+157 additions, -393 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/Library/ExclusiveLiveStream.cs` (+6 -0) 📝 `Emby.Server.Implementations/Library/MediaSourceManager.cs` (+16 -17) 📝 `Emby.Server.Implementations/LiveTv/EmbyTV/DirectRecorder.cs` (+12 -4) 📝 `Emby.Server.Implementations/LiveTv/TunerHosts/HdHomerun/HdHomerunUdpStream.cs` (+1 -6) 📝 `Emby.Server.Implementations/LiveTv/TunerHosts/LiveStream.cs` (+22 -95) 📝 `Emby.Server.Implementations/LiveTv/TunerHosts/SharedHttpStream.cs` (+15 -34) 📝 `Jellyfin.Api/Controllers/LiveTvController.cs` (+3 -3) 📝 `Jellyfin.Api/Controllers/VideosController.cs` (+8 -12) 📝 `Jellyfin.Api/Helpers/AudioHelper.cs` (+11 -14) ➖ `Jellyfin.Api/Helpers/ProgressiveFileCopier.cs` (+0 -187) 📝 `Jellyfin.Api/Helpers/ProgressiveFileStream.cs` (+24 -11) 📝 `Jellyfin.Api/Models/StreamingDtos/StreamState.cs` (+3 -0) ➕ `MediaBrowser.Controller/Library/IDirectStreamProvider.cs` (+19 -0) 📝 `MediaBrowser.Controller/Library/ILiveStream.cs` (+3 -0) 📝 `MediaBrowser.Controller/Library/IMediaSourceManager.cs` (+14 -10) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** I was going to just fix the seeking thing but ended up doing a tiny bit more. It's maybe too much at once... **Issues** Fixes #5724 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:30:51 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#11012