Merge pull request #69 from thefringeninja/structured-logs

Use Structured Logging

Author: thefringeninja

src/SqlStreamStore.Server/Browser/SqlStreamStoreBrowserMiddleware.cs

@@ -22,7 +22,8 @@ public static IApplicationBuilder UseSqlStreamStoreBrowser(
                 .Where(name => name.StartsWith(typeof(SqlStreamStoreBrowserMiddleware).Namespace));
 
             Log.Debug(
-                $"The following embedded resources were found and will be served as static content: {string.Join(", ", staticFiles)}");
+                "The following embedded resources were found and will be served as static content: {staticFiles}", 
+                string.Join(", ", staticFiles));
 
             return builder.Use(IndexPage).UseStaticFiles(new StaticFileOptions
             {

src/SqlStreamStore.Server/SqlStreamStoreFactory.cs

@@ -46,7 +46,7 @@ public Task<IStreamStore> Create(CancellationToken cancellationToken = default)
             var provider = _configuration.Provider?.ToLowerInvariant()
                            ?? inmemory;
 
-            Log.Information($"Creating stream store for provider '{provider}'");
+            Log.Information("Creating stream store for provider {provider}", provider);
 
             if (!s_factories.TryGetValue(provider, out var factory))
             {
@@ -170,13 +170,14 @@ async Task<bool> DatabaseExists()
         private static void SchemaCreationFailed(Func<string> getSchemaCreationScript, Exception ex)
             => Log.Warning(
                 new StringBuilder()
-                    .Append($"Could not create schema: {ex.Message}")
+                    .Append("Could not create schema: {ex}")
                     .AppendLine()
                     .Append(
                         "Does your connection string have enough permissions? If not, run the following sql script as a privileged user:")
                     .AppendLine()
-                    .Append(getSchemaCreationScript())
+                    .Append("{script}")
                     .ToString(),
-                ex);
+                ex,
+                getSchemaCreationScript());
     }
 }