fix: update code

Signed-off-by: Junjie Gao <[email protected]>
Azure · Sep 5, 2023 · e10c491 · e10c491
1 parent 8720cb5
commit e10c491
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/Notation.Plugin.AzureKeyVault/Certificate/CertificateChain.cs b/Notation.Plugin.AzureKeyVault/Certificate/CertificateChain.cs
@@ -82,19 +82,19 @@ 2. non-self-signed leaf cert
             var leafCert = certs.First(x => !caSet.Contains(x.SubjectName.Name));
 
             // build the certificate chain
-            X509Certificate2Collection chain = new X509Certificate2Collection();
+            var chain = new X509Certificate2Collection();
             var currentCert = leafCert;
             while (true)
             {
                 chain.Add(currentCert);
-                var subjectDN = currentCert.SubjectName.Name;
-                var issuerDN = currentCert.IssuerName.Name;
 
                 if (isRootCA(currentCert))
                 {
                     break;
                 }
 
+                var subjectDN = currentCert.SubjectName.Name;
+                var issuerDN = currentCert.IssuerName.Name;
                 if (!caSet.Remove(issuerDN))
                 {
                     throw new PluginException($"Found multiple certificates issued by {issuerDN}");
@@ -119,9 +119,6 @@ 2. non-self-signed leaf cert
         /// </summary>
         /// <param name="cert"></param>
         /// <returns></returns>
-        public static bool isRootCA(X509Certificate2 cert)
-        {
-            return cert.SubjectName.Name == cert.IssuerName.Name;
-        }
+        public static bool isRootCA(X509Certificate2 cert) => cert.SubjectName.Name == cert.IssuerName.Name;
     }
 }