it-swarm.com.ru

Весенний ботинок REST @RequestParam не проверяется

Я попробовал несколько примеров из сети и не могу заставить Spring проверить мой параметр строки запроса. Это не похоже на выполнение REGEX/сбой.

package my.controller;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestParam;
import org.springframework.web.bind.annotation.RestController;

import javax.validation.Valid;
import javax.validation.constraints.Pattern;

import static org.springframework.web.bind.annotation.RequestMethod.GET;

@RestController
public class MyController {

    private static final String VALIDATION_REGEX = "^[0-9]+(,[0-9]+)*$";

    @RequestMapping(value = "/my/{id}", method = GET)
    public myResonseObject getMyParams(@PathVariable("id") String id,
                                       @Valid @Pattern(regexp = VALIDATION_REGEX) 
                                       @RequestParam(value = "myparam", required = true) String myParam) {
         // Do Stuff!
    }

}

Текущее поведение

PASS - /my/1?myparam=1
PASS - /my/1?myparam=1,2,3
PASS - /my/1?myparam=
PASS - /my/1?myparam=1,bob

Желаемое поведение

PASS - /my/1?myparam=1
PASS - /my/1?myparam=1,2,3
FAIL - /my/1?myparam=
FAIL - /my/1?myparam=1,bob

Спасибо

10
ptimson

Вам нужно добавить @Validated в ваш класс следующим образом:

@RestController
@Validated
class Controller {
  // ...
}

ОБНОВЛЕНИЕ:

вам нужно правильно его настроить .. добавьте этот bean в ваш контекст:

@Bean
 public MethodValidationPostProcessor methodValidationPostProcessor() {
      return new MethodValidationPostProcessor();
 }

Пример для обработки исключения :

@ControllerAdvice
@Component
public class GlobalExceptionHandler {
    @ExceptionHandler
    @ResponseBody
    @ResponseStatus(HttpStatus.BAD_REQUEST)
    public Map handle(MethodArgumentNotValidException exception) {
        return error(exception.getBindingResult().getFieldErrors()
                .stream()
                .map(FieldError::getDefaultMessage)
                .collect(Collectors.toList()));
    }


    @ExceptionHandler
    @ResponseBody
    @ResponseStatus(HttpStatus.BAD_REQUEST)
    public Map handle(ConstraintViolationException exception) {
        return error(exception.getConstraintViolations()
                .stream()
                .map(ConstraintViolation::getMessage)
                .collect(Collectors.toList()));
    }

    private Map error(Object message) {
        return Collections.singletonMap("error", message);
    }
}
29
Jaiwo99

Вы можете попробовать это

@Pattern(regexp="^[0-9]+(,[0-9]+)*$")
private static final String VALIDATION_REGEX;

(обратите внимание на окончательный модификатор) или еще 

 @Pattern()
 private static final String VALIDATION_REGEX = "^[0-9]+(,[0-9]+)*$";

А затем удалите @Pattern (regexp = VALIDATION_REGEX) из вашего метода и оставьте только аннотацию @Valid

public myResonseObject getMyParams(@PathVariable("id") String id, @Valid @RequestParam(value = "myparam", required = true) String myParam) {
1
georges van

У вас неправильное регулярное выражение

"^[0-9]+(,[0-9]+)*$"

Никогда не будет разбирать

1,bob

Может быть, вам нужно:

"^\w+(,\w+)*$"

И если вам нужно проанализировать также пустую строку, используйте:

"^(\w+(,\w+)*)?$"
0
Gangnus