fix: no violation logged on blocking validation

Author: pboos

spring-boot-starter/spring-boot-starter-web/src/main/java/com/getyourguide/openapi/validation/filter/OpenApiValidationInterceptor.java

@@ -150,7 +150,9 @@ private List<OpenApiViolation> validateRequest(
                 .validateRequestObjectAsync(requestMetaData, responseMetaData, requestBody, openApiViolationHandler);
             return List.of();
         } else {
-            return validator.validateRequestObject(requestMetaData, requestBody);
+            var violations = validator.validateRequestObject(requestMetaData, requestBody);
+            violations.forEach(openApiViolationHandler::onOpenApiViolation);
+            return violations;
         }
     }
 
@@ -185,7 +187,9 @@ private List<OpenApiViolation> validateResponse(
                 .validateResponseObjectAsync(requestMetaData, responseMetaData, responseBody, openApiViolationHandler);
             return List.of();
         } else {
-            return validator.validateResponseObject(requestMetaData, responseMetaData, responseBody);
+            var violations = validator.validateResponseObject(requestMetaData, responseMetaData, responseBody);
+            violations.forEach(openApiViolationHandler::onOpenApiViolation);
+            return violations;
         }
     }
 

spring-boot-starter/spring-boot-starter-webflux/src/main/java/com/getyourguide/openapi/validation/filter/OpenApiValidationWebFilter.java

@@ -141,7 +141,9 @@ private List<OpenApiViolation> validateRequest(
         }
 
         if (runType == RunType.SYNC) {
-            return validator.validateRequestObject(requestMetaData, responseMetaData, request.getCachedBody());
+            var violations = validator.validateRequestObject(requestMetaData, responseMetaData, request.getCachedBody());
+            violations.forEach(openApiViolationHandler::onOpenApiViolation);
+            return violations;
         } else {
             validator.validateRequestObjectAsync(
                 requestMetaData, responseMetaData, request.getCachedBody(), openApiViolationHandler);
@@ -160,7 +162,9 @@ private List<OpenApiViolation> validateResponse(
         }
 
         if (runType == RunType.SYNC) {
-            return validator.validateResponseObject(requestMetaData, responseMetaData, responseBody);
+            var violations = validator.validateResponseObject(requestMetaData, responseMetaData, responseBody);
+            violations.forEach(openApiViolationHandler::onOpenApiViolation);
+            return violations;
         } else {
             validator
                 .validateResponseObjectAsync(requestMetaData, responseMetaData, responseBody, openApiViolationHandler);