Reputation: 3631
I have the following controller method:
@RequestMapping(value="/map/update", method=RequestMethod.POST, produces = "application/json; charset=utf-8")
@ResponseBody
public ResponseEntityWrapper updateMapTheme(
HttpServletRequest request,
@RequestBody @Valid List<CompanyTag> categories,
HttpServletResponse response
) throws ResourceNotFoundException, AuthorizationException {
...
}
CompanyTag is defined this way:
public class CompanyTag {
@StringUUIDValidation String key;
String value;
String color;
String icon;
Icon iconObj;
public String getKey() {
return key;
}
public void setKey(String key) {
this.key = key;
}
...
}
The problem is that validation is not triggered, the CompanyTag list is not validated, the "StringUUIDValidation" validator is never called.
If I remove the List and only try to send a single CompanyTag, i.e. instead of:
@RequestBody @Valid List<CompanyTag> categories,
use:
@RequestBody @Valid CompanyTag category,
it works as expected, so apparently Spring does not like to validate lists of things (tried with array instead, that did not work either).
Anybody have any idea what's missing?
Upvotes: 93
Views: 192773
Reputation: 1
@RequestBody List<@Valid CompanyTag> categories;
instead of focusing on @Valid List<?>,
focus on the object itself like List<@Valid ?>
.
Upvotes: 0
Reputation: 3697
With the later versions of spring, you can now do this.
@RequestMapping(value="/map/update", method=RequestMethod.POST, produces = "application/json; charset=utf-8")
@ResponseBody
public ResponseEntityWrapper updateMapTheme(
HttpServletRequest request,
@RequestBody List<@Valid CompanyTag> categories,
HttpServletResponse response
) throws ResourceNotFoundException, AuthorizationException {
...
}
the @Valid annotation is in the generic param.
If you are using a custom javax validation annotation, make sure to add TYPE_USE to the annotation target
@Target({ ElementType.TYPE_USE})
public @interface ValidationAnnotation {.. }
Upvotes: 5
Reputation: 31
You need to add @Validated to the Controller class.
If you're interested in what's happening behind the scenes in the framework, here's a brief explanation.(My SpringBoot version is 3.2.1):
When the @Validated annotation is added to a controller class in the Spring framework, Spring employs its Aspect-Oriented Programming (AOP) capabilities to enact the annotation's functionality. This is achieved through the use of the org.springframework.validation.beanvalidation.MethodValidationInterceptor interceptor. This particular interceptor is tasked with intercepting calls to methods within the controller. Once a method call is intercepted, the interceptor utilizes the jakarta.validation.executable.ExecutableValidator's validateParameters method. This method is instrumental in validating method parameters, including those of the List type. Thus, the interceptor ensures that the parameters passed to controller methods adhere to the defined validation constraints.
Upvotes: 1
Reputation: 636
After several tries, I figured it out this solution! Hope will be useful for you.
Verify! are you pointing to Java 17 from build path (Java 17 or higher, as Spring Boot 3.x is used which brings Hibernate-Validator 8.0.0.Final)
@PostMapping
@ResponseStatus(code = HttpStatus.OK)
public List<CompanyTag> updateMapTheme(@RequestBody List<@Valid CompanyTag> companyTag) {
return appService.updateMapTheme(companyTag);
}
Make sure that @Valid should be inside diamond brackets! @RequestBody List<@Valid Event> events
Finally! Restart the IDE once you change the JDK Path to 17 or higher.
Cheers! Happy coding :)
Upvotes: 1
Reputation: 809
To add to the above by @laffuste with Lombok, in Spring Boot 2.7, I have a MyDtoList validator that delegates back down to the singular validator for a plural argument. In my Spring RestController that has singular and plural arguments:
In application yaml:
spring:
jackson:
deserialization:
accept-single-value-as-array: true
In my controller:
@InitBinder("myDto")
public void addMyDtoValidator(WebDataBinder binder) {
binder.addValidators(myDtoValidator);
}
@InitBinder("myDtoList")
public void addMyDtoListValidator(WebDataBinder binder) {
binder.addValidators(myDtoListValidator);
}
Then the validator code:
private MyDtoValidator singleDtoValidator;
public MyDtoListValidator(MyDtoValidator singleDtoValidator) {
this.singleDtoValidator = singleDtoValidator;
}
@Override
public boolean supports(Class<?> clazz) {
return ValidList.class.isAssignableFrom(clazz);
}
@Override
public void validate(Object target, Errors errors) {
if (target == null) {
errors.rejectValue(null, "should not be null error");
} else {
ValidList<MyDto> list = (ValidList<MyDto>) target;
for (MyDtodto: list) {
singleDtoValidator.validate(dto, errors);
}
}
}
Upvotes: 0
Reputation: 11
I have done custom Validation for the list of parameters we're passing... `
import java.lang.annotation.Documented;
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;
import javax.validation.Constraint;
import javax.validation.Payload;
@Target({ElementType.PARAMETER})
@Retention(RetentionPolicy.RUNTIME)
@Documented
@Constraint(validatedBy = PatternListValidator.class)
public @interface PatternList {
public String regexp();
public String message() default "Invalid inputs";
public Class<?>[] groups() default {};
Class<? extends Payload>[] payload() default {};
}`
Created the above custom validation annotation / interface and implemented the same with the business logic
import java.util.List;
import javax.validation.ConstraintValidator;
import javax.validation.ConstraintValidatorContext;
public class PatternListValidator implements ConstraintValidator<PatternList, List<String>> {
private String regexp;
@Override
public void initialize(PatternList patternList) {
this.regexp = patternList.regexp();
}
@Override
public boolean isValid(List<String> dataList, ConstraintValidatorContext context) {
for(String data : dataList) {
if(!data.matches(regexp)) {
return false;
}
}
return true;
}
}
used this @PatternList annotation in my controller class as api method parameter as below
Public ResponseEntity<Object> getStudents(
@ApiParam(name = "studentIds", value = "Fetch students for athlete and art. Example values: 1234, 5432", required = true) @PatternList(regexp = "\\d+", message = "student Id's can contain only numbers") @RequestParam(value = "studentId", required = true) List<String> studentIds) {
business logic goes here....
}
Upvotes: 0
Reputation: 140
I am using
I needed to validate a List<String>
request parameters. Here is my working example (inspired by some of previous answers)
@RestController
@Validated
class SearchController {
@GetMapping("/search")
fun search(
@Valid
@RequestParam(value = "term") terms: List<Term>,
): Mono<ResponseEntity<SearchResponse>> {...}
}
data class Term(
@field:NotEmpty(
message = "Term is required"
)
@field:Size(
min = 2,
max = 500,
message = "Term length out of range"
)
val term: String
)
in build.gradle.kts
dependencies {
implementation("org.springframework.boot:spring-boot-starter-validation")
}
Upvotes: 1
Reputation: 619
I did the below steps to make validation work on lists:
@Validated
at the class levelList<@Valid MyClass>
Also, found that if the validation failed I got javax.validation.ConstraintViolationException
Upvotes: 9
Reputation: 11
For those using spring boot (I was using 2.6.7), what worked for me was adding the spring-boot-starter-validation dependency:
org.springframework.boot:spring-boot-starter-validation
Upvotes: 1
Reputation: 6517
Using Spring Boot 2.4.1
:
Add the @Validated
annotation to the class
Move the @Valid
annotation inside the diamond operator:
@RestController
@Validated // <-- This activates the Spring Validation AOP interceptor
public class MyController {
...
@RequestBody List<@Valid CompanyTag> categories
// ^^^ - NOTE: the @Valid annotation is inside <> brackets
Upvotes: 27
Reputation: 193
With the Spring Boot 2.2.2 version...
Here's the piece of code:-
import java.util.List;
import javax.validation.Valid;
import javax.validation.constraints.NotBlank;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RestController;
@RestController
@Validated
public class MyController {
@PostMapping(value = "/test", consumes = "application/json", produces = "application/json")
public String test(@Valid @RequestBody List<Student> st) {
System.out.println("-------------test Method-------");
return "Its' Success";
}
}
class Student{
@NotBlank
String name;
@NotBlank
String password;
@NotBlank
String email;
public String getName() {
return name;
}
public void setName(String name) {
this.name = name;
}
public String getPassword() {
return password;
}
public void setPassword(String password) {
this.password = password;
}
public String getEmail() {
return email;
}
public void setEmail(String email) {
this.email = email;
}
}
List of JSON Data:-
Notice name is blank in the second Student object.
[
{
"name": "Sreepad",
"password": "sddwh",
"email": "[email protected]"
},
{
"name": "",
"password": "sddwh",
"email": "[email protected]"
}
]
Error Description:-
javax.validation.ConstraintViolationException: test.st[1].name: must not be blank.
Note: List and String won't be validated at method parameter level if you remove @Validated at Class level.
17. Validation
The method validation feature supported by Bean Validation 1.1 is automatically enabled as long as a JSR-303 implementation (such as Hibernate validator) is on the classpath. This lets bean methods be annotated with javax.validation constraints on their parameters and/or on their return value. Target classes with such annotated methods need to be annotated with the @Validated annotation at the type level for their methods to be searched for inline constraint annotations.
Upvotes: 0
Reputation: 9881
(this answer is in Kotlin, for Java see https://stackoverflow.com/a/64061936)
For those using kotlin and jackson, here is the ValidatedList
class that do not require wrapping, that is, it will still be serialized/deserialized as a usual list:
class ValidatedList<E> {
/**
* By default, spring-boot cannot validate lists, as they are generic AND do not conform to the Java Bean definition.
* This is one work-around: create a wrapper that fits the Java Bean definition, and use Jackson annotations to
* make the wrapper disappear upon (de)serialization.
* Do not change anything (such as making the _value field private) or it won't work anymore !
*
* Usage:
* ```
* @PostMapping("/something")
* fun someRestControllerMethod(@Valid @RequestBody pojoList: ValidatedList<SomePOJOClass>){
* // access list with:
* pojoList.values
*}
* ```
*/
@JsonValue
@Valid
@NotNull
@Size(min = 1, message = "array body must contain at least one item.")
var _values: List<E>? = null
val values: List<E>
get() = _values!!
@JsonCreator
constructor(vararg list: E) {
this._values = list.asList()
}
}
Advantages:
@Validated
annotation@Size
)400 Bad Request
(which is not the case when using javax
and @Validated
annotation)Example:
data class N(
@field:Min(value = 0, message = "id must be positive.")
val id: Long? = null,
@field:NotNull
@field:Size(min = 2, max = 32, message = "wrong size: should be 32 chars long.")
val token: String? = null
)
@RestController
class XController {
@PostMapping("/ns")
fun getNs(@Valid @NotNull @RequestBody wrap: ListWrapper<N>) = wrap
}
Submit ok:
curl -H "Content-Type: application/json" -X POST http://localhost:8080/ns -d '[{"id": 11, "token": "something"}]'
[{"id" : 11, "token" : "something"}]
Submit empty body:
curl -H "Content-Type: application/json" -X POST http://localhost:8080/ns -d '[]'
{
"timestamp" : "2020-09-25T08:49:30.324+00:00",
"message" : "Validation failed for object='listWrapper'. Error count: 1",
"error" : "Bad Request",
"path" : "/ns",
"status" : 400,
"exception" : "org.springframework.web.bind.MethodArgumentNotValidException",
"trace":"org.springframework.web.bind.MethodArgumentNotValidException: Validation failed for argument [0] in public com.example.demo.test.XController$ListWrapper<com.example.demo.test.XController$N> com.example.demo.test.XController.getNs(com.example.demo.test.XController$ListWrapper<com.example.demo.test.XController$N>): [Field error in object 'listWrapper' on field '_values': rejected value [[]]; codes [Size.listWrapper._values,Size._values,Size.java.util.List,Size]; [...]"
}
Submit invalid items:
curl -H "Content-Type: application/json" -X POST http://localhost:8080/ns -d '[{"id": -11, "token": ""}]'
{
"message" : "Validation failed for object='listWrapper'. Error count: 2",
"path" : "/ns",
"exception" : "org.springframework.web.bind.MethodArgumentNotValidException",
"timestamp" : "2020-09-25T08:49:54.505+00:00",
"error" : "Bad Request",
"status" : 400,
"trace":"org.springframework.web.bind.MethodArgumentNotValidException: Validation failed for argument [0] in public com.example.demo.test.XController$ListWrapper<com.example.demo.test.XController$N> com.example.demo.test.XController.getNs(com.example.demo.test.XController$ListWrapper<com.example.demo.test.XController$N>) with 2 errors: [...]"
}
Upvotes: 0
Reputation: 2878
I tried to use Paul's method in my project, but some people said it's too complex. Not long after that, I find another easy way which works like code below:
@Validated
@RestController
@RequestMapping("/parent")
public class ParentController {
private FatherRepository fatherRepository;
/**
* DI
*/
public ParentController(FatherRepository fatherRepository) {
this.fatherRepository = fatherRepository;
}
@PostMapping("/test")
public void test(@RequestBody @Valid List<Father> fathers) {
}
}
It works and easy to use. The key point is the @Valiated annotation on the class. Btw, it's springBootVersion = '2.0.4.RELEASE' that I use.
As discussed in comments, exceptions can be handled like code below:
@RestControllerAdvice
@Component
public class ControllerExceptionHandler {
/**
* handle controller methods parameter validation exceptions
*
* @param exception ex
* @return wrapped result
*/
@ExceptionHandler
@ResponseBody
@ResponseStatus(HttpStatus.OK)
public DataContainer handle(ConstraintViolationException exception) {
Set<ConstraintViolation<?>> violations = exception.getConstraintViolations();
StringBuilder builder = new StringBuilder();
for (ConstraintViolation<?> violation : violations) {
builder.append(violation.getMessage());
break;
}
DataContainer container = new DataContainer(CommonCode.PARAMETER_ERROR_CODE, builder.toString());
return container;
}
}
Taking http status code as representing network is ok and only first violation message is returned here. You may change it to satisfy customized requirements.
With @Validated on class level, parameters of methods are validated by what called method-level validation in spring boot, which is not only worked for controllers, but any bean the IOC
container managed.
By the way, the methods in method level validation (short as validation A) is enhanced by
while the typical spring boot controller methods validation (short as validation B) is processed in
Both of them lead the actual validation operation to org.hibernate.validator.internal.engine.ValidatorImpl
by default, but the methods they call are different, which leads to the differences in validation logic.
MethodValidationInterceptor
call validateParameters
method in ValidatorImpl
RequestResponseBodyMethodProcessor
call validate
method in ValidatorImpl
They are different methods with different functions, so lead to different results in validation A/B, the typical point is the validation of list object:
The JSR-303 defines functions of the methods we discussed above.
validate
method is explained in the validation method part, and the implementation must obey the logic defined in validation routine, in which it states that it will execute all the constraint validation for all reachable fields of the object, this is why element of List
object (or other collection instance) cannot be validated via this method - the elements of the collection are not fields of the collection instance.
But validateParameters
, JSR-303 actually doesn't treat it as main topic and put it in Appendix C. Proposal for method-level validation
. It provides some description:
The constraints declarations evaluated are the constraints hosted on the parameters of the method or constructor. If @Valid is placed on a parameter, constraints declared on the object itself are considered.
validateReturnedValue evaluates the constraints hosted on the method itself. If @Valid is placed on the method, the constraints declared on the object itself are considered.
public @NotNull String saveItem(@Valid @NotNull Item item, @Max(23) BigDecimal price)
In the previous example,
- item is validated against @NotNull and all the constraints it hosts
- price is validated against @Max(23)
- the result of saveItem is validated against @NotNull
and exclaim that Bean Validation providers are free to implement this proposal as a specific extension
. As far as I know, the Hibernate Validation
project implements this method, makes constraints works on the object itself, and element of collection object.
I don't know why the spring framework guys call validate
in RequestResponseBodyMethodProcessor
, makes lots of related questions appeare in stackoverflow. Maybe it's just because http post body data usually is a form data, and can be represented by a java bean naturally. If it's me, I'll call the validateParametes
in RequestResponseBodyMethodProcessor
for easy use.
Upvotes: 72
Reputation: 1653
Here's my attempt to reconcile the many different answers.
Lebecca's answer works without the need for a wrapper, as Paul's answer requires, because @Validated
placed on the class enables the method validation feature of the Bean Validation API.
The Hibernate Validator documentation specifically explains:
[...] the @Valid annotation can be used to mark executable parameters and return values for cascaded validation.
[...]
Cascaded validation can not only be applied to simple object references but also to collection-typed parameters and return values. This means when putting the @Valid annotation to a parameter or return value which
is an array
implements java.lang.Iterable
or implements java.util.Map
each contained element gets validated.
If you need to validate a collection of Beans, this is the most convenient way (make sure to also implement an @ExceptionHandler
as required).
If you need to validate a collection of Non-Beans, e.g. a List<String>
where each element must match a pattern, you can use container element constraints like this:
controllerMethod(List<@Pattern(regexp="pattern") String> strings)
There's also the possibility to only use @Valid
on a controller method parameter (which must then be a Bean type) without also placing @Validated
on the class. In that case, you get an appropriate, detailed HTTP 400 response "for free", i.e. without the need for a custom @ExceptionHandler
. But this doesn't apply the cascading validation, so you cannot validate something like @Valid List<SomeBean> beans
, nor does it support container element constraints.
And finally, you can combine the latter approach with an extra parameter added to the method of type BindingResult
. This won't trigger an automatic error response in the case of a validation error, but instead you must inspect the injected BindingResult
yourself in the method body and act accordingly (which allows for more flexibility). That is described in this comprehensive answer.
Upvotes: 4
Reputation: 4295
The @Valid
annotation can be used inside the diamond operator:
private List<@Valid MyType> types;
or
@Valid
private List<MyType> types;
Now, every list item will be validated.
Upvotes: 6
Reputation: 21
create entity class:
import javax.validation.Valid;
import java.util.List;
public class ValidList<E> {
@Valid
private List<E> list;
public List<E> getList() {
return list;
}
public void setList(List<E> list) {
this.list = list;
}
}
use Controller
@RequestMapping(value = "/sku", method = RequestMethod.POST)
public JsonResult createSKU(@Valid @RequestBody ValidList<Entity> entityList, BindingResult bindingResult) {
if (bindingResult.hasErrors())
return ErrorTools.build().handlerError(bindingResult);
return new JsonResult(200, "result");
}
Upvotes: 1
Reputation: 159
use @Validated annotate controller
use @Valid annotate @RequestBody
Upvotes: 4
Reputation: 5713
I'm using spring-boot 1.5.19.RELEASE
I annotate my service with @validated
and then apply @Valid
to the List
parameter in the method and items in my list get validated.
Model
@Data
@ApiModel
@Validated
public class SubscriptionRequest {
@NotBlank()
private String soldToBpn;
@NotNull
@Size(min = 1)
@Valid
private ArrayList<DataProducts> dataProducts;
private String country;
@NotNull
@Size(min = 1)
@Valid
private ArrayList<Contact> contacts;
}
Service Interface (or use on concrete type if no interface)
@Validated
public interface SubscriptionService {
List<SubscriptionCreateResult> addSubscriptions(@NonNull @Size(min = 1) @Valid List<SubscriptionRequest> subscriptionRequestList)
throws IOException;
}
Global Exception Handler method (ApiError Type is not my design)
@ResponseStatus(HttpStatus.BAD_REQUEST)
@ExceptionHandler(value = ConstraintViolationException.class)
@ResponseBody
public ApiError[] handleConstraintViolationException(ConstraintViolationException exception) {
List<InvalidField> invalidFields = exception.getConstraintViolations().stream()
.map(constraintViolation -> new InvalidField(constraintViolation.getPropertyPath().toString(),
constraintViolation.getMessage(),
constraintViolation.getInvalidValue()))
.collect(Collectors.toList());
return new ApiError[] {new ApiError(ErrorCodes.INVALID_PARAMETER, "Validation Error", invalidFields)};
}
example bad method call from a controller
LinkedList<SubscriptionRequest> list = new LinkedList<>();
list.add(new SubscriptionRequest());
return subscriptionService.addSubscriptions(list);
Response body (note the index [0])
[
{
"errorCode": "invalid.parameter",
"errorMessage": "Validation Error",
"invalidFields": [
{
"name": "addSubscriptions.arg0[0].soldToBpn",
"message": "may not be empty",
"value": null
},
{
"name": "addSubscriptions.arg0[0].dataProducts",
"message": "may not be null",
"value": null
},
{
"name": "addSubscriptions.arg0[0].contacts",
"message": "may not be null",
"value": null
}
]
}
]
Upvotes: 3
Reputation: 17105
@Paul Strack's great solution mixed with Lombok magic:
@Data
public class ValidList<E> implements List<E> {
@Valid
@Delegate
private List<E> list = new ArrayList<>();
}
Usage (swap List for ValidList):
public ResponseEntityWrapper updateMapTheme(
@RequestBody @Valid ValidList<CompanyTag> categories, ...)
(Needs Lombok, but if you don't use it already you really want to try it out)
Upvotes: 33
Reputation: 1973
I think the most elegant solution is to create a custom Validator for Collection and a @ControllerAdvice that registers that Validator in the WebDataBinders, take a look to Spring validation for RequestBody parameters bound to collections in Controller methods
Upvotes: 7
Reputation: 813
I found another approach that works. The basic problem is that you want to have a list as your input payload for your service, but javax.validation won't validate a list, only a JavaBean. The trick is to use a custom list class that functions as both a List and a JavaBean:
@RequestBody @Valid List<CompanyTag> categories
Change to:
@RequestBody @Valid ValidList<CompanyTag> categories
Your list subclass would look something like this:
public class ValidList<E> implements List<E> {
@Valid
private List<E> list;
public ValidList() {
this.list = new ArrayList<E>();
}
public ValidList(List<E> list) {
this.list = list;
}
// Bean-like methods, used by javax.validation but ignored by JSON parsing
public List<E> getList() {
return list;
}
public void setList(List<E> list) {
this.list = list;
}
// List-like methods, used by JSON parsing but ignored by javax.validation
@Override
public int size() {
return list.size();
}
@Override
public boolean isEmpty() {
return list.isEmpty();
}
// Other list methods ...
}
Upvotes: 78
Reputation: 7646
I would suggest to wrap your List categories into some DTO bean and validate it. Beside of working validation you will benefit from more flexible API.
@RequestMapping(value="/map/update", method=RequestMethod.POST, produces = "application/json; charset=utf-8")
@ResponseBody
public ResponseEntityWrapper updateMapTheme(
HttpServletRequest request,
@RequestBody @Valid TagRequest tagRequest,
HttpServletResponse response
) throws ResourceNotFoundException, AuthorizationException {
...
}
public static class TagRequest {
@Valid
List<CompanyTag> categories;
// Gettes setters
}
Upvotes: 15
Reputation: 1489
Validating a collection does not work directly.
For example: what should it do if multiple elements fail the validation? Stop after first validation? Validate all (if so what is to be done with the collection of messages)?
If in your configuration Spring delegates to a Bean Validator provider like Hibernate Validator, you should look up for ways of implementing a collection validator there.
For Hibernate, a similar problem is discussed here
Upvotes: 4