From fd0f887e418b314f5bd9d11de7d07505a6ca34cd Mon Sep 17 00:00:00 2001 From: Todd Martin Date: Mon, 7 Nov 2022 15:19:54 -0500 Subject: [PATCH] Added logging on Captcha error. (cherry picked from commit d1ed4fed8a12434e360a690ce5e5d8c6448b11e1) --- .../Implementations/HCaptchaValidationService.cs | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/Core/Services/Implementations/HCaptchaValidationService.cs b/src/Core/Services/Implementations/HCaptchaValidationService.cs index b8a63c642c..54682333a7 100644 --- a/src/Core/Services/Implementations/HCaptchaValidationService.cs +++ b/src/Core/Services/Implementations/HCaptchaValidationService.cs @@ -74,9 +74,10 @@ public class HCaptchaValidationService : ICaptchaValidationService _logger.LogError(11389, e, "Unable to verify with HCaptcha."); return response; } - + if (!responseMessage.IsSuccessStatusCode) { + _logger.LogError("Error submitting Captcha token to HCaptcha: {status} - {message}", responseMessage.StatusCode, responseMessage.ReasonPhrase); return response; } @@ -86,6 +87,12 @@ public class HCaptchaValidationService : ICaptchaValidationService response.MaybeBot = score >= _globalSettings.Captcha.MaybeBotScoreThreshold; response.IsBot = score >= _globalSettings.Captcha.IsBotScoreThreshold; response.Score = score; + + if (!response.Success && hcaptchaResponse.ErrorCodes != null && hcaptchaResponse.ErrorCodes.Any()) + { + _logger.LogError("HCaptcha errors received when validating Captcha for user {email}: {@errors}", user.Email, hcaptchaResponse.ErrorCodes); + } + return response; } @@ -125,6 +132,8 @@ public class HCaptchaValidationService : ICaptchaValidationService public double? Score { get; set; } [JsonPropertyName("score_reason")] public List ScoreReason { get; set; } + [JsonPropertyName("error-codes")] + public List ErrorCodes {get;set;} public void Dispose() { } }