reword comments

Author: mauroservienti

src/ServiceControl.Audit.Persistence.RavenDB/MemoryInformationRetriever.cs

@@ -8,9 +8,11 @@ namespace ServiceControl.Audit.Persistence.RavenDB;
 
 class MemoryInformationRetriever(DatabaseConfiguration databaseConfiguration)
 {
-    // TODO what does a connection string look like? Is it only a URI or could it contain other stuff?
-    //   The ?? operator is needed because ServerUrl is populated when running embedded and connection string when running in external mode.
-    //   However the tricky part is that when tests are run they behave like if it was external mode
+    // What does a connection string look like? Is it only a URI or could it contain other stuff?
+    // The ?? operator is needed because ServerUrl is populated when running embedded and connection
+    // string when running in external mode. However, the tricky part is that when tests are run they
+    // behave like if it was external mode. If the connection string contain always only the server
+    // URL, this code is safe, otherwise it need to be adjusted to extract the server URL.
     readonly HttpClient client = new() { BaseAddress = new Uri(databaseConfiguration.ServerConfiguration.ServerUrl ?? databaseConfiguration.ServerConfiguration.ConnectionString) };
 
     record ResponseDto

src/ServiceControl.Persistence.RavenDB/MemoryInformationRetriever.cs

@@ -8,7 +8,10 @@ namespace ServiceControl.Persistence.RavenDB;
 
 class MemoryInformationRetriever(RavenPersisterSettings persisterSettings)
 {
-    // TODO what does a connection string look like? Is it only a URI or could it contain other stuff?
+    // What does a connection string look like? Is it only a URI or could it contain other stuff?
+    // The primary instance has only the concept of a connection string (vs the Audit instance having
+    // both a ServiceUrl and a ConnectionString). If the connection string contain always only the
+    // server URL, this code is safe, otherwise it need to be adjusted to extract the server URL. 
     readonly HttpClient client = new() { BaseAddress = new Uri(persisterSettings.ConnectionString) };
 
     record ResponseDto