Merge pull request 'Added inline logging for critical bugs' (#23) from payments into master

Reviewed-on: #23

MidrandBooksApi/Payments/Payfast/PayfastConfirmationEndpoint.cs

@@ -41,7 +41,7 @@ public sealed class PayfastConfirmationEndpoint : IEndpoint
 
             if (signatureCheck.IsFailed || !string.Equals(signatureCheck.Value, incomingSignature, StringComparison.OrdinalIgnoreCase))
             {
-                logger.LogCritical("Incoming sugnature failed validation: {signature}, {errors}", incomingSignature, signatureCheck.Errors.Select(e => e.Message).ToList());
+                logger.LogCritical($"Incoming sugnature failed validation: {incomingSignature}, {signatureCheck.Errors.Select(e => e.Message).ToList()}");
 
                 return Results.Unauthorized();
             }
@@ -56,7 +56,7 @@ public sealed class PayfastConfirmationEndpoint : IEndpoint
 
             if (serverConfirmation.IsFailed || !serverConfirmation.Value)
             {
-                logger.LogCritical("Server confirmation failed: {rawstring}, {errors}", rawQueryParamString, serverConfirmation.Errors.Select(e => e.Message).ToList());
+                logger.LogCritical($"Server confirmation failed: {rawQueryParamString}, {serverConfirmation.Errors.Select(e => e.Message).ToList()}");
 
                 return Results.Unauthorized();
             }