diff --git a/pom.xml b/pom.xml index 33ff93e95..5e45e318c 100644 --- a/pom.xml +++ b/pom.xml @@ -532,7 +532,7 @@ com.diffplug.spotless spotless-maven-plugin - 2.33.0 + 2.38.0 diff --git a/src/main/java/org/owasp/webgoat/lessons/authbypass/AccountVerificationHelper.java b/src/main/java/org/owasp/webgoat/lessons/authbypass/AccountVerificationHelper.java index 41b64d518..4de8d5ef7 100644 --- a/src/main/java/org/owasp/webgoat/lessons/authbypass/AccountVerificationHelper.java +++ b/src/main/java/org/owasp/webgoat/lessons/authbypass/AccountVerificationHelper.java @@ -42,6 +42,7 @@ public class AccountVerificationHelper { static { secQuestionStore.put(verifyUserId, userSecQuestions); } + // end 'data store set up' // this is to aid feedback in the attack process and is not intended to be part of the @@ -68,6 +69,7 @@ public class AccountVerificationHelper { return likely; } + // end of cheating check ... the method below is the one of real interest. Can you find the flaw? public boolean verifyAccount(Integer userId, HashMap submittedQuestions) {