[PR #4125] [MERGED] Networking 2 (Cumulative PR) - Swapping over to new NetworkManager #9896

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4125
Author: @BaronGreenback
Created: 9/12/2020
Status: Merged
Merged: 11/22/2020
Merged by: @joshuaboniface

Base: masterHead: NetworkPR2


📝 Commits (10+)

📊 Changes

34 files changed (+1075 additions, -1232 deletions)

View changed files

📝 CONTRIBUTORS.md (+1 -0)
📝 Emby.Dlna/Main/DlnaEntryPoint.cs (+25 -15)
📝 Emby.Dlna/PlayTo/PlayToManager.cs (+1 -9)
📝 Emby.Dlna/Server/DescriptionXmlBuilder.cs (+1 -17)
📝 Emby.Server.Implementations/ApplicationHost.cs (+100 -181)
📝 Emby.Server.Implementations/Emby.Server.Implementations.csproj (+0 -1)
📝 Emby.Server.Implementations/EntryPoints/ExternalPortForwarding.cs (+7 -4)
📝 Emby.Server.Implementations/LiveTv/LiveTvMediaSourceProvider.cs (+1 -2)
📝 Emby.Server.Implementations/LiveTv/TunerHosts/HdHomerun/HdHomerunUdpStream.cs (+24 -1)
Emby.Server.Implementations/Networking/NetworkManager.cs (+0 -566)
📝 Emby.Server.Implementations/Udp/UdpServer.cs (+1 -1)
📝 Jellyfin.Api/Controllers/DlnaServerController.cs (+1 -1)
📝 Jellyfin.Api/Controllers/StartupController.cs (+5 -3)
📝 Jellyfin.Api/Controllers/SystemController.cs (+4 -4)
Jellyfin.Api/Helpers/ClassMigrationHelper.cs (+71 -0)
Jellyfin.Networking/Manager/INetworkManager.cs (+0 -234)
📝 Jellyfin.Server/CoreAppHost.cs (+0 -3)
📝 Jellyfin.Server/Extensions/ApiApplicationBuilderExtensions.cs (+3 -2)
📝 Jellyfin.Server/Middleware/BaseUrlRedirectionMiddleware.cs (+2 -1)
📝 Jellyfin.Server/Middleware/IpBasedAccessValidationMiddleware.cs (+21 -20)

...and 14 more files

📄 Description

The new networkManager working with Jellyfin. Lots of functionality isn't available yet.

This was the minimum amount of changes for a successful compilation and execution.

Depends on PR #4124 and PR #4253

To be used in conjunction with Web change jellyfin/jellyfin-web#1979


🔄 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/4125 **Author:** [@BaronGreenback](https://github.com/BaronGreenback) **Created:** 9/12/2020 **Status:** ✅ Merged **Merged:** 11/22/2020 **Merged by:** [@joshuaboniface](https://github.com/joshuaboniface) **Base:** `master` ← **Head:** `NetworkPR2` --- ### 📝 Commits (10+) - [`9ef79d1`](https://github.com/jellyfin/jellyfin/commit/9ef79d190b2490a03c566bfaaf963fbba7d124a9) Large number of files - [`288d894`](https://github.com/jellyfin/jellyfin/commit/288d89493e76b8881c719fe549859b2c0b5f7e29) Fixed testing units. - [`b44455a`](https://github.com/jellyfin/jellyfin/commit/b44455ad0d6e78b5baed535c06a7f7d49116d1ee) Update based on PR1 changes. - [`38b8110`](https://github.com/jellyfin/jellyfin/commit/38b8110a3ec33267bb2b9e0b75ed0e2a704f41ca) Removing blank lines. - [`c1bfba9`](https://github.com/jellyfin/jellyfin/commit/c1bfba90813b6780d43f82d4056496f5c776a186) Nullable added - [`b04aed2`](https://github.com/jellyfin/jellyfin/commit/b04aed2f5842f6dbdc07c5d8d44f04bb69e0896d) Merge remote-tracking branch 'upstream/master' into NetworkPR2 - [`40464a6`](https://github.com/jellyfin/jellyfin/commit/40464a6fddf3b522c57a72e16bd036fbe8ac4580) Update LanFilteringMiddleware.cs - [`96c3c4a`](https://github.com/jellyfin/jellyfin/commit/96c3c4af4237c1fd1955fe5840904015ae592bef) Update IpBasedAccessValidationMiddleware.cs - [`e6a3518`](https://github.com/jellyfin/jellyfin/commit/e6a3518220f159325cdf941fbfbbcd8ba871c28c) Uploading latest - [`0bc37c9`](https://github.com/jellyfin/jellyfin/commit/0bc37c981d63a4d8168d622c452d6fac1b061e07) Merge branch 'NetworkPR2' of https://github.com/BaronGreenback/jellyfin into NetworkPR2 ### 📊 Changes **34 files changed** (+1075 additions, -1232 deletions) <details> <summary>View changed files</summary> 📝 `CONTRIBUTORS.md` (+1 -0) 📝 `Emby.Dlna/Main/DlnaEntryPoint.cs` (+25 -15) 📝 `Emby.Dlna/PlayTo/PlayToManager.cs` (+1 -9) 📝 `Emby.Dlna/Server/DescriptionXmlBuilder.cs` (+1 -17) 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+100 -181) 📝 `Emby.Server.Implementations/Emby.Server.Implementations.csproj` (+0 -1) 📝 `Emby.Server.Implementations/EntryPoints/ExternalPortForwarding.cs` (+7 -4) 📝 `Emby.Server.Implementations/LiveTv/LiveTvMediaSourceProvider.cs` (+1 -2) 📝 `Emby.Server.Implementations/LiveTv/TunerHosts/HdHomerun/HdHomerunUdpStream.cs` (+24 -1) ➖ `Emby.Server.Implementations/Networking/NetworkManager.cs` (+0 -566) 📝 `Emby.Server.Implementations/Udp/UdpServer.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/DlnaServerController.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/StartupController.cs` (+5 -3) 📝 `Jellyfin.Api/Controllers/SystemController.cs` (+4 -4) ➕ `Jellyfin.Api/Helpers/ClassMigrationHelper.cs` (+71 -0) ➖ `Jellyfin.Networking/Manager/INetworkManager.cs` (+0 -234) 📝 `Jellyfin.Server/CoreAppHost.cs` (+0 -3) 📝 `Jellyfin.Server/Extensions/ApiApplicationBuilderExtensions.cs` (+3 -2) 📝 `Jellyfin.Server/Middleware/BaseUrlRedirectionMiddleware.cs` (+2 -1) 📝 `Jellyfin.Server/Middleware/IpBasedAccessValidationMiddleware.cs` (+21 -20) _...and 14 more files_ </details> ### 📄 Description The new networkManager working with Jellyfin. Lots of functionality isn't available yet. This was the minimum amount of changes for a successful compilation and execution. Depends on PR #4124 and PR #4253 To be used in conjunction with Web change jellyfin/jellyfin-web#1979 --- <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:10:24 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9896