Merge pull request #99 from it-at-m/fix-captcha-metrics

Fabinatix97 · web-flow · commit 8294dac994f5 · 2025-09-12T15:26:54.000+02:00
Add debug logging
diff --git a/captchaservice-backend/src/main/java/de/muenchen/captchaservice/service/captcha/CaptchaService.java b/captchaservice-backend/src/main/java/de/muenchen/captchaservice/service/captcha/CaptchaService.java
@@ -58,10 +58,13 @@ public boolean verify(final String siteKey, final ExtendedPayload payload, final
         }
         try {
             final boolean isValid = Altcha.verifySolution(payload, captchaProperties.hmacKey(), true);
+            log.info("Altcha.verifySolution() returned {} for payloadHash={}", isValid, getPayloadHash(payload));
             if (isValid) {
+                log.info("Sucessfully verified. Recording success for metrics...");
                 metricsService.recordVerifySuccess(siteKey, sourceAddress);
 
                 if (payload.getTook() != null) {
+                    log.info("Recording client solve time metric: took={} ms", payload.getTook());
                     metricsService.recordClientSolveTime(siteKey, sourceAddress, payload.getTook());
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -58,10 +58,13 @@ public boolean verify(final String siteKey, final ExtendedPayload payload, final`
`58`	`58`	`}`
`59`	`59`	`try {`
`60`	`60`	`final boolean isValid = Altcha.verifySolution(payload, captchaProperties.hmacKey(), true);`
	`61`	`+ log.info("Altcha.verifySolution() returned {} for payloadHash={}", isValid, getPayloadHash(payload));`
`61`	`62`	`if (isValid) {`
	`63`	`+ log.info("Sucessfully verified. Recording success for metrics...");`
`62`	`64`	`metricsService.recordVerifySuccess(siteKey, sourceAddress);`
`63`	`65`
`64`	`66`	`if (payload.getTook() != null) {`
	`67`	`+ log.info("Recording client solve time metric: took={} ms", payload.getTook());`
`65`	`68`	`metricsService.recordClientSolveTime(siteKey, sourceAddress, payload.getTook());`
`66`	`69`	`}`
`67`	`70`