RavenDB-21872 Change GetBasicCertificateInfo return string format

haludi · Jan 2, 2024 · 65c0119 · 65c0119
1 parent b39bbfe
commit 65c0119
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Raven.Server/RavenServer.cs b/src/Raven.Server/RavenServer.cs
@@ -1257,7 +1257,7 @@ public async Task StartCertificateReplicationAsync(byte[] certBytes, bool replac
 
                 if (Logger.IsOperationsEnabled)
                 {
-                    Logger.Operations($"Starting certificate replication. current:{Certificate.Certificate.GetBasicCertificateInfo()}, new:{newCertificate.GetBasicCertificateInfo()}");
+                    Logger.Operations($"Starting certificate replication. current:\"{Certificate.Certificate.GetBasicCertificateInfo()}\", new:\"{newCertificate.GetBasicCertificateInfo()}\"");
                 }
 
                 // During replacement of a cluster certificate, we must have both the new and the old server certificates registered in the server store.

diff --git a/src/Raven.Server/Utils/CertificateUtils.cs b/src/Raven.Server/Utils/CertificateUtils.cs
@@ -619,7 +619,7 @@ private static X509Certificate2 BuildNewPfx(SetupInfo setupInfo, X509Certificate
 
         public static string GetBasicCertificateInfo(this X509Certificate2 certificate)
         {
-            return $"{{\"Thumbprint\":\"{certificate.Thumbprint}\", \"Subject\":\"{certificate.Subject}\"}}";
+            return $"Thumbprint: {certificate.Thumbprint}, Subject: {certificate.Subject}";
         }
     }
     public static class PublicKeyPinningHashHelpers