From a492082f4e015d6d38368c4ac05d39d236387214 Mon Sep 17 00:00:00 2001
From: Shadowghost <Ghost_of_Stone@web.de>
Date: Wed, 20 Jul 2022 11:47:48 +0200
Subject: Apply review suggestions and fix build

---
 MediaBrowser.Common/Net/IPData.cs            | 17 +++++------------
 MediaBrowser.Common/Net/NetworkExtensions.cs | 10 +++++-----
 2 files changed, 10 insertions(+), 17 deletions(-)

(limited to 'MediaBrowser.Common/Net')

diff --git a/MediaBrowser.Common/Net/IPData.cs b/MediaBrowser.Common/Net/IPData.cs
index 3c6adc7e86..6901d6ad8a 100644
--- a/MediaBrowser.Common/Net/IPData.cs
+++ b/MediaBrowser.Common/Net/IPData.cs
@@ -14,13 +14,12 @@ namespace MediaBrowser.Common.Net
         /// </summary>
         /// <param name="address">An <see cref="IPAddress"/>.</param>
         /// <param name="subnet">The <see cref="IPNetwork"/>.</param>
-        public IPData(
-            IPAddress address,
-            IPNetwork? subnet)
+        /// <param name="name">The object's name.</param>
+        public IPData(IPAddress address, IPNetwork? subnet, string name)
         {
             Address = address;
             Subnet = subnet ?? (address.AddressFamily == AddressFamily.InterNetwork ? new IPNetwork(address, 32) : new IPNetwork(address, 128));
-            Name = string.Empty;
+            Name = name;
         }
 
         /// <summary>
@@ -28,15 +27,9 @@ namespace MediaBrowser.Common.Net
         /// </summary>
         /// <param name="address">An <see cref="IPAddress"/>.</param>
         /// <param name="subnet">The <see cref="IPNetwork"/>.</param>
-        /// <param name="name">The object's name.</param>
-        public IPData(
-            IPAddress address,
-            IPNetwork? subnet,
-            string name)
+        public IPData(IPAddress address, IPNetwork? subnet)
+            : this(address, subnet, string.Empty)
         {
-            Address = address;
-            Subnet = subnet ?? (address.AddressFamily == AddressFamily.InterNetwork ? new IPNetwork(address, 32) : new IPNetwork(address, 128));
-            Name = name;
         }
 
         /// <summary>
diff --git a/MediaBrowser.Common/Net/NetworkExtensions.cs b/MediaBrowser.Common/Net/NetworkExtensions.cs
index 4cba1c99f2..ae1e47ccc2 100644
--- a/MediaBrowser.Common/Net/NetworkExtensions.cs
+++ b/MediaBrowser.Common/Net/NetworkExtensions.cs
@@ -13,6 +13,10 @@ namespace MediaBrowser.Common.Net
     /// </summary>
     public static class NetworkExtensions
     {
+        // Use regular expression as CheckHostName isn't RFC5892 compliant.
+        // Modified from gSkinner's expression at https://stackoverflow.com/questions/11809631/fully-qualified-domain-name-validation
+        private static readonly Regex _fqdnRegex = new Regex(@"(?im)^(?!:\/\/)(?=.{1,255}$)((.{1,63}\.){0,127}(?![0-9]*$)[a-z0-9-]+\.?)(:(\d){1,5}){0,1}$");
+
         /// <summary>
         /// Returns true if the IPAddress contains an IP6 Local link address.
         /// </summary>
@@ -227,12 +231,8 @@ namespace MediaBrowser.Common.Net
 
             if (hosts.Length <= 2)
             {
-                // Use regular expression as CheckHostName isn't RFC5892 compliant.
-                // Modified from gSkinner's expression at https://stackoverflow.com/questions/11809631/fully-qualified-domain-name-validation
-                string pattern = @"(?im)^(?!:\/\/)(?=.{1,255}$)((.{1,63}\.){0,127}(?![0-9]*$)[a-z0-9-]+\.?)(:(\d){1,5}){0,1}$";
-
                 // Is hostname or hostname:port
-                if (Regex.IsMatch(hosts[0], pattern))
+                if (_fqdnRegex.IsMatch(hosts[0]))
                 {
                     try
                     {
-- 
cgit v1.2.3