Add .NET 10 target and make use of C#14 extension members (#1672)

* Add .NET 10 target

* fix IDE0031

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0031

* fix ca5399

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca5399

* fix ca1515

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1515

* fix ca2002

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca2002

* fix ca1508

new false positives.

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1508

* fix ca2000

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca2000

* fix ca2025

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca2025

* fix ca1849

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1849

* fix Reverse() overloads

because of https://learn.microsoft.com/en-us/dotnet/core/compatibility/core-libraries/10.0/csharp-overload-resolution

* supress CA2002

* Use extension members for ThrowHelpers

* use extension members for CryptoAbstractions

* use extension member for DateTime.UnixEpoch

* use extension members for string.Join etc

* use extension members for Convert.To/FromHexString

* disable CA1508

* Update .NET 10 RC2

* Workaround Build Regression in .NET 10 RC2

dotnet/sdk#51265

* suppress new warnings introduced by merge

* Update to .NET 10 final release

* Revert "Workaround Build Regression in .NET 10 RC2"

This is fixed in the final release.

This reverts commit 5a59ac9.

* fix new warnings with MSTest 4 + .NET 10

* use same Randomizer instance

* disable CA2000

* reduce CA1849 suppressions

and disable duplicate S6966

* disable preview analyzers

reverts 6c3c06d

Author: mus65

.editorconfig

@@ -177,7 +177,7 @@ dotnet_diagnostic.S2699.severity = none
 # S2930: "IDisposables" should be disposed
 # https://rules.sonarsource.com/csharp/RSPEC-2930/
 #
-# Duplicate of CA2000.
+# too noisy.
 dotnet_diagnostic.S2930.severity = none
 
 # S2933: Fields that are only assigned in the constructor should be "readonly"
@@ -344,6 +344,10 @@ dotnet_diagnostic.S5659.severity = none
 # https://rules.sonarsource.com/csharp/RSPEC-5773/
 dotnet_diagnostic.S4581.severity = none
 
+# S6966: Awaitable method should be used
+# Duplicate of CA1849
+dotnet_diagnostic.S6966.severity = none
+
 #### StyleCop rules ####
 
 # SA1003: Symbols must be spaced correctly
@@ -683,6 +687,10 @@ dotnet_diagnostic.CA1305.severity = none
 # https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1309
 dotnet_diagnostic.CA1309.severity = none
 
+# CA1508: Avoid dead conditional code
+# Too many false positives.
+dotnet_diagnostic.CA1508.severity = none
+
 # CA1510: Use ArgumentNullException throw helper
 #
 # This is only available in .NET 6.0 and higher. We'd need to use conditional compilation to only
@@ -726,6 +734,14 @@ dotnet_diagnostic.CA1848.severity = silent
 # By default, this diagnostic is only reported for private members.
 dotnet_code_quality.CA1859.api_surface = private,internal
 
+# CA1873: Evaluation of this argument may be expensive and unnecessary if logging is disabled
+dotnet_diagnostic.CA1873.severity = suggestion
+
+# CA2000: Dispose objects before losing scope
+#
+# too noisy.
+dotnet_diagnostic.CA2000.severity = suggestion
+
 # CA2208: Instantiate argument exceptions correctly
 # https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca2208
 #

.github/workflows/build.yml

@@ -18,33 +18,33 @@ jobs:
       uses: actions/setup-dotnet@v5
 
     - name: Build Unit Tests .NET
-      run: dotnet build -f net9.0 test/Renci.SshNet.Tests/
+      run: dotnet build -f net10.0 test/Renci.SshNet.Tests/
 
     - name: Build IntegrationTests .NET
-      run: dotnet build -f net9.0 test/Renci.SshNet.IntegrationTests/
+      run: dotnet build -f net10.0 test/Renci.SshNet.IntegrationTests/
 
     - name: Run Unit Tests .NET
       run: |
         dotnet test \
-          -f net9.0 \
+          -f net10.0 \
           --no-build \
           --logger "console;verbosity=normal" \
           --logger GitHubActions \
           -p:CollectCoverage=true \
           -p:CoverletOutputFormat=cobertura \
-          -p:CoverletOutput=../../coverlet/linux_unit_test_net_9_coverage.xml \
+          -p:CoverletOutput=../../coverlet/linux_unit_test_net_10_coverage.xml \
           test/Renci.SshNet.Tests/
 
     - name: Run Integration Tests .NET
       run: |
         dotnet test \
-          -f net9.0 \
+          -f net10.0 \
           --no-build \
           --logger "console;verbosity=normal" \
           --logger GitHubActions \
           -p:CollectCoverage=true \
           -p:CoverletOutputFormat=cobertura \
-          -p:CoverletOutput=../../coverlet/linux_integration_test_net_9_coverage.xml \
+          -p:CoverletOutput=../../coverlet/linux_integration_test_net_10_coverage.xml \
           test/Renci.SshNet.IntegrationTests/
 
     - name: Archive Coverlet Results
@@ -82,13 +82,13 @@ jobs:
     - name: Run Unit Tests .NET
       run: |
         dotnet test `
-          -f net9.0 `
+          -f net10.0 `
           --no-build `
           --logger "console;verbosity=normal" `
           --logger GitHubActions `
           -p:CollectCoverage=true `
           -p:CoverletOutputFormat=cobertura `
-          -p:CoverletOutput=../../coverlet/windows_unit_test_net_9_coverage.xml `
+          -p:CoverletOutput=../../coverlet/windows_unit_test_net_10_coverage.xml `
           test/Renci.SshNet.Tests/
 
     - name: Run Unit Tests .NET Framework
@@ -173,7 +173,7 @@ jobs:
     - name: Run Integration Tests .NET
       run:
         dotnet test `
-          -f net9.0 `
+          -f net10.0 `
           --logger "console;verbosity=normal" `
           --logger GitHubActions `
           -p:CollectCoverage=true `

Directory.Build.props

@@ -23,7 +23,6 @@
     -->
   <PropertyGroup>
     <EnableNETAnalyzers>true</EnableNETAnalyzers>
-    <AnalysisLevel>preview-All</AnalysisLevel>
     <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
   </PropertyGroup>
 

Directory.Packages.props

@@ -13,7 +13,7 @@
     <!-- Should stay on LTS .NET releases. -->
     <PackageVersion Include="Microsoft.Bcl.Cryptography" Version="10.0.0" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
-    <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="9.0.9" />
+    <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="10.0.0" />
     <PackageVersion Include="MSTest" Version="4.0.2" />
     <PackageVersion Include="Moq" Version="4.20.72" />
     <PackageVersion Include="Nerdbank.GitVersioning" Version="3.7.115" />

global.json

@@ -1,6 +1,6 @@
 {
   "sdk": {
-    "version": "9.0.300",
+    "version": "10.0.100",
     "rollForward": "latestFeature"
   }
 }

src/Renci.SshNet/Abstractions/ConvertExtensions.cs

@@ -0,0 +1,34 @@
+#nullable enable
+#if !NET
+using System.Text;
+#endif
+
+namespace System
+{
+    internal static class ConvertExtensions
+    {
+        extension(Convert)
+        {
+#if !NET
+            public static byte[] FromHexString(string s)
+            {
+                return Org.BouncyCastle.Utilities.Encoders.Hex.Decode(s);
+            }
+
+            public static string ToHexString(byte[] inArray)
+            {
+                ArgumentNullException.ThrowIfNull(inArray);
+
+                var builder = new StringBuilder(inArray.Length * 2);
+
+                foreach (var b in inArray)
+                {
+                    builder.Append(b.ToString("X2"));
+                }
+
+                return builder.ToString();
+            }
+#endif
+        }
+    }
+}

src/Renci.SshNet/Abstractions/CryptoAbstraction.cs

@@ -7,80 +7,8 @@ namespace Renci.SshNet.Abstractions
 {
     internal static class CryptoAbstraction
     {
-        private static readonly RandomNumberGenerator Randomizer = RandomNumberGenerator.Create();
+        internal static readonly RandomNumberGenerator Randomizer = RandomNumberGenerator.Create();
 
         internal static readonly SecureRandom SecureRandom = new SecureRandom(new CryptoApiRandomGenerator(Randomizer));
-
-        /// <summary>
-        /// Generates a <see cref="byte"/> array of the specified length, and fills it with a
-        /// cryptographically strong random sequence of values.
-        /// </summary>
-        /// <param name="length">The length of the array generate.</param>
-        public static byte[] GenerateRandom(int length)
-        {
-            var random = new byte[length];
-            Randomizer.GetBytes(random);
-            return random;
-        }
-
-        public static byte[] HashMD5(byte[] source)
-        {
-#if NET
-            return MD5.HashData(source);
-#else
-            using (var md5 = MD5.Create())
-            {
-                return md5.ComputeHash(source);
-            }
-#endif
-        }
-
-        public static byte[] HashSHA1(byte[] source)
-        {
-#if NET
-            return SHA1.HashData(source);
-#else
-            using (var sha1 = SHA1.Create())
-            {
-                return sha1.ComputeHash(source);
-            }
-#endif
-        }
-
-        public static byte[] HashSHA256(byte[] source)
-        {
-#if NET
-            return SHA256.HashData(source);
-#else
-            using (var sha256 = SHA256.Create())
-            {
-                return sha256.ComputeHash(source);
-            }
-#endif
-        }
-
-        public static byte[] HashSHA384(byte[] source)
-        {
-#if NET
-            return SHA384.HashData(source);
-#else
-            using (var sha384 = SHA384.Create())
-            {
-                return sha384.ComputeHash(source);
-            }
-#endif
-        }
-
-        public static byte[] HashSHA512(byte[] source)
-        {
-#if NET
-            return SHA512.HashData(source);
-#else
-            using (var sha512 = SHA512.Create())
-            {
-                return sha512.ComputeHash(source);
-            }
-#endif
-        }
     }
 }

src/Renci.SshNet/Abstractions/DateTimeExtensions.cs

@@ -0,0 +1,19 @@
+#nullable enable
+namespace System
+{
+    internal static class DateTimeExtensions
+    {
+        extension(DateTime)
+        {
+#if !NET
+            public static DateTime UnixEpoch
+            {
+                get
+                {
+                    return new DateTime(1970, 1, 1, 0, 0, 0, 0, DateTimeKind.Utc);
+                }
+            }
+#endif
+        }
+    }
+}

src/Renci.SshNet/Abstractions/MD5Extensions.cs

@@ -0,0 +1,19 @@
+#nullable enable
+namespace System.Security.Cryptography
+{
+    internal static class MD5Extensions
+    {
+        extension(MD5)
+        {
+#if !NET
+            public static byte[] HashData(byte[] source)
+            {
+                using (var md5 = MD5.Create())
+                {
+                    return md5.ComputeHash(source);
+                }
+            }
+#endif
+        }
+    }
+}

src/Renci.SshNet/Abstractions/RandomNumberGeneratorExtensions.cs

@@ -0,0 +1,22 @@
+#nullable enable
+#if !NET
+using Renci.SshNet.Abstractions;
+#endif
+
+namespace System.Security.Cryptography
+{
+    internal static class RandomNumberGeneratorExtensions
+    {
+        extension(RandomNumberGenerator)
+        {
+#if !NET
+            public static byte[] GetBytes(int length)
+            {
+                var random = new byte[length];
+                CryptoAbstraction.Randomizer.GetBytes(random);
+                return random;
+            }
+#endif
+        }
+    }
+}