Merge pull request 'refactored incoming signature validator to use form fields instead of httprequest' (#111) from payments into master
Reviewed-on: #111
This commit was merged in pull request #111.
This commit is contained in:
@@ -48,13 +48,8 @@ public sealed partial class PayfastService(IDbContextFactory<MidrandBooksDbConte
|
||||
}
|
||||
}
|
||||
|
||||
public static bool VerifyIncomingSignature(HttpRequest request, string passphrase)
|
||||
public static bool VerifyIncomingSignature(IDictionary<string, string> formFields, string passphrase)
|
||||
{
|
||||
var formFields = new Dictionary<string, string>(StringComparer.Ordinal);
|
||||
|
||||
foreach (var file in request.Form)
|
||||
formFields.Add(file.Key, file.Value.ToString());
|
||||
|
||||
if (!formFields.TryGetValue("signature", out string? incomingSignature))
|
||||
return false;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user