Encapsulate token extraction into validator

Particular · Aug 30, 2023 · 1074c5e · 1074c5e
1 parent 728676d
commit 1074c5e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/NServiceBus.Core/Hosting/Helpers/AssemblyScanner.cs b/src/NServiceBus.Core/Hosting/Helpers/AssemblyScanner.cs
@@ -374,8 +374,7 @@ bool ShouldScanDependencies(Assembly assembly)
                 return false;
             }
 
-            var tokenString = Convert.ToHexString(assemblyName.GetPublicKeyToken() ?? Array.Empty<byte>());
-            if (AssemblyValidator.IsRuntimeAssembly(tokenString))
+            if (AssemblyValidator.IsRuntimeAssembly(assemblyName))
             {
                 return false;
             }

diff --git a/src/NServiceBus.Core/Hosting/Helpers/AssemblyValidator.cs b/src/NServiceBus.Core/Hosting/Helpers/AssemblyValidator.cs
@@ -2,6 +2,7 @@
 {
     using System;
     using System.IO;
+    using System.Reflection;
     using System.Reflection.Metadata;
     using System.Reflection.PortableExecutable;
     using System.Security.Cryptography;
@@ -47,6 +48,12 @@ public static void ValidateAssemblyFile(string assemblyPath, out bool shouldLoad
             reason = "File is a .NET assembly.";
         }
 
+        public static bool IsRuntimeAssembly(AssemblyName assemblyName)
+        {
+            var tokenString = Convert.ToHexString(assemblyName.GetPublicKeyToken() ?? Array.Empty<byte>());
+            return IsRuntimeAssembly(tokenString);
+        }
+
         static string GetPublicKeyToken(byte[] publicKey)
         {
             using var sha1 = SHA1.Create();
@@ -58,7 +65,7 @@ static string GetPublicKeyToken(byte[] publicKey)
             return Convert.ToHexString(lastEightBytes);
         }
 
-        public static bool IsRuntimeAssembly(string tokenString) =>
+        static bool IsRuntimeAssembly(string tokenString) =>
             tokenString switch
             {
                 // Microsoft tokens