Merge pull request #38 from rarimo/fix/verify_passport

Fix proof country validation
rarimo · Jun 27, 2024 · 68aa707 · 68aa707
2 parents 1970264 + b0ae0d9
commit 68aa707
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/internal/service/handlers/verify_passport.go b/internal/service/handlers/verify_passport.go
@@ -98,9 +98,17 @@ func VerifyPassport(w http.ResponseWriter, r *http.Request) {
 			balAID = *balance.AnonymousID
 		}
 
+		proofCountry, err := requests.ExtractCountry(*proof)
+		if err != nil {
+			log.WithError(err).Error("failed to extract country while proof was successfully verified")
+			ape.RenderErr(w, problems.InternalError())
+			return
+		}
+
 		err = validation.Errors{
-			"data/attributes/country":      validation.Validate(*balance.Country, validation.Required, validation.In(country)),
-			"data/attributes/anonymous_id": validation.Validate(anonymousID, validation.Required, validation.In(balAID)),
+			"data/attributes/country":                   validation.Validate(*balance.Country, validation.Required, validation.In(country)),
+			"data/attributes/anonymous_id":              validation.Validate(anonymousID, validation.Required, validation.In(balAID)),
+			"data/attributes/proof/pub_signals/country": validation.Validate(proofCountry, validation.Required, validation.In(*balance.Country)),
 		}.Filter()
 		if err != nil {
 			ape.RenderErr(w, problems.BadRequest(err)...)