diff options
| author | Shadowghost <Ghost_of_Stone@web.de> | 2022-10-07 09:04:41 +0200 |
|---|---|---|
| committer | Shadowghost <Ghost_of_Stone@web.de> | 2022-10-07 09:16:02 +0200 |
| commit | 7b90fcd0531e1fd414df404b58f2f05b0641be01 (patch) | |
| tree | 9cb45933f68308a9dca1e6217318742f3ef7f9d6 /Jellyfin.Networking/Manager | |
| parent | bd9a940fed129d99fe9ffedafec324e795549c90 (diff) | |
| parent | 719e5eae16a3488de449a5b2a7c56132c8f1e5c1 (diff) | |
Merge branch 'master' into network-rewrite
Diffstat (limited to 'Jellyfin.Networking/Manager')
| -rw-r--r-- | Jellyfin.Networking/Manager/NetworkManager.cs | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/Jellyfin.Networking/Manager/NetworkManager.cs b/Jellyfin.Networking/Manager/NetworkManager.cs index 146340989b..333afd7484 100644 --- a/Jellyfin.Networking/Manager/NetworkManager.cs +++ b/Jellyfin.Networking/Manager/NetworkManager.cs @@ -77,8 +77,11 @@ namespace Jellyfin.Networking.Manager #pragma warning disable CS8618 // Non-nullable field is uninitialized. : Values are set in UpdateSettings function. Compiler doesn't yet recognise this. public NetworkManager(IConfigurationManager configurationManager, ILogger<NetworkManager> logger) { - _logger = logger ?? throw new ArgumentNullException(nameof(logger)); - _configurationManager = configurationManager ?? throw new ArgumentNullException(nameof(configurationManager)); + ArgumentNullException.ThrowIfNull(logger); + ArgumentNullException.ThrowIfNull(configurationManager); + + _logger = logger; + _configurationManager = configurationManager; _initLock = new(); _interfaces = new List<IPData>(); _macAddresses = new List<PhysicalAddress>(); @@ -497,7 +500,9 @@ namespace Jellyfin.Networking.Manager /// <param name="configuration">The <see cref="NetworkConfiguration"/> to use.</param> public void UpdateSettings(object configuration) { - NetworkConfiguration config = (NetworkConfiguration)configuration ?? throw new ArgumentNullException(nameof(configuration)); + ArgumentNullException.ThrowIfNull(configuration); + + var config = (NetworkConfiguration)configuration; InitialiseLan(config); InitialiseRemote(config); @@ -826,10 +831,7 @@ namespace Jellyfin.Networking.Manager /// <inheritdoc/> public bool IsInLocalNetwork(IPAddress address) { - if (address == null) - { - throw new ArgumentNullException(nameof(address)); - } + ArgumentNullException.ThrowIfNull(address); // See conversation at https://github.com/jellyfin/jellyfin/pull/3515. if (TrustAllIpv6Interfaces && address.AddressFamily == AddressFamily.InterNetworkV6) |
