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

Added inline logging for critical bugs #23

Merged
khwezi merged 1 commits from payments into master 2026-06-13 15:30:29 +02:00
Conversation 0 Commits 1 Files Changed 1
+2 -2
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)) 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(); return Results.Unauthorized();
} }
@@ -56,7 +56,7 @@ public sealed class PayfastConfirmationEndpoint : IEndpoint
if (serverConfirmation.IsFailed || !serverConfirmation.Value) 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(); return Results.Unauthorized();
} }