[PR #4124] [MERGED] Networking: 1 - Network Manager #9893

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4124
Author: @BaronGreenback
Created: 9/12/2020
Status: Merged
Merged: 11/21/2020
Merged by: @Bond-009

Base: masterHead: NetworkPR


📝 Commits (10+)

  • 6edc746 NetworkManager
  • 91da8cd Config file with additional options.
  • 4abf6cf Update Jellyfin.Networking/Manager/INetworkManager.cs
  • b64cbc6 Update Jellyfin.Networking/Manager/INetworkManager.cs
  • 71d3f47 Update MediaBrowser.Model/Configuration/ServerConfiguration.cs
  • 8f80858 Update MediaBrowser.Model/Configuration/ServerConfiguration.cs
  • c04e5dc Update MediaBrowser.Model/Configuration/ServerConfiguration.cs
  • 43ccef6 Update Jellyfin.Networking/Manager/NetworkManager.cs
  • 5120bd2 Split function.
  • 6e87a1c Merge branch 'NetworkPR' of https://github.com/BaronGreenback/jellyfin into NetworkPR

📊 Changes

15 files changed (+3532 additions, -210 deletions)

View changed files

📝 Emby.Server.Implementations/Networking/NetworkManager.cs (+20 -10)
📝 Jellyfin.Api/Controllers/StartupController.cs (+3 -3)
Jellyfin.Networking/Configuration/NetworkConfiguration.cs (+221 -0)
Jellyfin.Networking/Configuration/NetworkConfigurationExtensions.cs (+21 -0)
Jellyfin.Networking/Configuration/NetworkConfigurationFactory.cs (+27 -0)
Jellyfin.Networking/Jellyfin.Networking.csproj (+30 -0)
Jellyfin.Networking/Manager/INetworkManager.cs (+234 -0)
Jellyfin.Networking/Manager/NetworkManager.cs (+1319 -0)
MediaBrowser.Common/Net/IPHost.cs (+445 -0)
MediaBrowser.Common/Net/IPNetAddress.cs (+277 -0)
MediaBrowser.Common/Net/IPObject.cs (+406 -0)
MediaBrowser.Common/Net/NetworkExtensions.cs (+262 -0)
MediaBrowser.Model/Configuration/PathSubstitution.cs (+20 -0)
📝 MediaBrowser.Model/Configuration/ServerConfiguration.cs (+227 -183)
📝 MediaBrowser.sln (+20 -14)

📄 Description

New PR's - same code. Been working on the simplification of the order of installation.

INetworkManager.cs - network manager interface.
NetworkManager.cs - replacement network manager object.
ServerConfiguration.cs has the initial configs on the lines of the setting to aid reading. It has the additional settings.
PathSubstitution.cs - is extracted from ServerConfiguration.cs and is unaltered.


🔄 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/4124 **Author:** [@BaronGreenback](https://github.com/BaronGreenback) **Created:** 9/12/2020 **Status:** ✅ Merged **Merged:** 11/21/2020 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `NetworkPR` --- ### 📝 Commits (10+) - [`6edc746`](https://github.com/jellyfin/jellyfin/commit/6edc746d247741e44779dd19bdb811ac05fa6e5a) NetworkManager - [`91da8cd`](https://github.com/jellyfin/jellyfin/commit/91da8cd6a26fbff691fa8bfe9017c3c11e0af57e) Config file with additional options. - [`4abf6cf`](https://github.com/jellyfin/jellyfin/commit/4abf6cfeb31122a9fd1ec1c5c888211e5de79a08) Update Jellyfin.Networking/Manager/INetworkManager.cs - [`b64cbc6`](https://github.com/jellyfin/jellyfin/commit/b64cbc608359eeb2793541abdd3a12b77ce47194) Update Jellyfin.Networking/Manager/INetworkManager.cs - [`71d3f47`](https://github.com/jellyfin/jellyfin/commit/71d3f4753a951b2210c3a5502e4d77ba3b5b490c) Update MediaBrowser.Model/Configuration/ServerConfiguration.cs - [`8f80858`](https://github.com/jellyfin/jellyfin/commit/8f808580394469febee5dfcd7827ed28302f7231) Update MediaBrowser.Model/Configuration/ServerConfiguration.cs - [`c04e5dc`](https://github.com/jellyfin/jellyfin/commit/c04e5dc29611a1700a12757df0bbd44c714ee9f5) Update MediaBrowser.Model/Configuration/ServerConfiguration.cs - [`43ccef6`](https://github.com/jellyfin/jellyfin/commit/43ccef63be8f5c1f1132b580fb2856d896d7b8cb) Update Jellyfin.Networking/Manager/NetworkManager.cs - [`5120bd2`](https://github.com/jellyfin/jellyfin/commit/5120bd2c9c19b5ef25e5e6321b980809ca2285e4) Split function. - [`6e87a1c`](https://github.com/jellyfin/jellyfin/commit/6e87a1ca1e61c624d7a3eb0d6a013e1b10135f99) Merge branch 'NetworkPR' of https://github.com/BaronGreenback/jellyfin into NetworkPR ### 📊 Changes **15 files changed** (+3532 additions, -210 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/Networking/NetworkManager.cs` (+20 -10) 📝 `Jellyfin.Api/Controllers/StartupController.cs` (+3 -3) ➕ `Jellyfin.Networking/Configuration/NetworkConfiguration.cs` (+221 -0) ➕ `Jellyfin.Networking/Configuration/NetworkConfigurationExtensions.cs` (+21 -0) ➕ `Jellyfin.Networking/Configuration/NetworkConfigurationFactory.cs` (+27 -0) ➕ `Jellyfin.Networking/Jellyfin.Networking.csproj` (+30 -0) ➕ `Jellyfin.Networking/Manager/INetworkManager.cs` (+234 -0) ➕ `Jellyfin.Networking/Manager/NetworkManager.cs` (+1319 -0) ➕ `MediaBrowser.Common/Net/IPHost.cs` (+445 -0) ➕ `MediaBrowser.Common/Net/IPNetAddress.cs` (+277 -0) ➕ `MediaBrowser.Common/Net/IPObject.cs` (+406 -0) ➕ `MediaBrowser.Common/Net/NetworkExtensions.cs` (+262 -0) ➕ `MediaBrowser.Model/Configuration/PathSubstitution.cs` (+20 -0) 📝 `MediaBrowser.Model/Configuration/ServerConfiguration.cs` (+227 -183) 📝 `MediaBrowser.sln` (+20 -14) </details> ### 📄 Description New PR's - same code. Been working on the simplification of the order of installation. INetworkManager.cs - network manager interface. NetworkManager.cs - replacement network manager object. ServerConfiguration.cs has the initial configs on the lines of the setting to aid reading. It has the additional settings. PathSubstitution.cs - is extracted from ServerConfiguration.cs and is unaltered. --- <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:22 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9893