litecharms/midrandbooks-api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 32 Wiki Activity

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

Browse Source 
Reviewed-on: #23
This commit was merged in pull request #23.
This commit is contained in:
Khwezi Mngoma
2026-06-13 15:30:28 +02:00
parent 8cc4425dfb c3981fd859
commit 01d3aa90ea
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
MidrandBooksApi/Payments/Payfast/PayfastConfirmationEndpoint.cs
+2 -2
View File
@@ -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();
}