Some checks for validator

Huluti · Nov 24, 2024 · a9c7f4b · a9c7f4b
1 parent 7f4d8ed
commit a9c7f4b
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/Validator/AltchaValidator.php b/src/Validator/AltchaValidator.php
@@ -40,11 +40,19 @@ public function validate($value, Constraint $constraint): void
 
         $request = $this->requestStack->getCurrentRequest();
         $altchaEncoded = $request->request->get('altcha');
+        if (!is_string($altchaEncoded)) {
+            $this->context->buildViolation($constraint->message)
+                ->addviolation();
+                return;
+        }
         $altchaJson = base64_decode($altchaEncoded, true);
+        if (!is_string($altchaJson)) {
+            $this->context->buildViolation($constraint->message)
+                ->addviolation();
+                return;
+        }
         $payload = json_decode($altchaJson, true);
 
-        // TODO: verify each line above
-
         if (!Altcha::verifySolution($payload, $this->hmacKey, true)) {
             $this->context->buildViolation($constraint->message)
                 ->addviolation();