0

Здравствуйте. Пытаюсь сделать регистрацию через spring security.

Вот метод контроллера:

@RequestMapping(value = "/registration", method = RequestMethod.POST)
    public ResponseEntity<String> registration(@RequestBody User newUser, BindingResult bindingResult) {
        User user = userService.save(newUser);
        if (bindingResult.hasErrors()) {
            return ResponseEntity.status(HttpStatus.BAD_REQUEST).build();
        }
        return ResponseEntity.status(HttpStatus.CREATED).build();
    }

Сервис:

  @Override
    public User save(User user) {
            user.setPassword(passwordEncoder.encode(user.getPassword()));
            Set<Role> roles = new HashSet<>();
            roles.add(roleRepository.getOne(1L));
            user.setRoles(roles);
            User newUser = userRepository.save(user);
            return newUser;
    }

Запрос делаю в json. Когда Есть username и password, то получаю 201

введите сюда описание изображения

Если нету или username, или password, то приходит 500

введите сюда описание изображения

Как сделать так, чтобы вместо 500 приходила 400-ошибка? Потому что насколько знаю, 500-ошибка не очень приветствуется.

1
  • Потому что насколько знаю, 500-ошибка не очень приветствуется. Это кто вам такое сказал? Никогда больше не говорите такого другим людям)) эта два разных типа ошибок, каждый с которых имеет свой полноценный скоуп юзабилити.
    – calm27
    23 янв 2018 в 13:21

2 ответа 2

1

Ваша ошибка выпадате по той причине, что эти поля в вашем объекте (@RequestBody User newUser) есть required. Вы можете сделать их optional в самом классе User или сделать проверку на null и если вместо какого-то value пришел null тогда возвращать свою ошибку return HttpStatuc.BAD_REQUEST (к примеру, это зависит, какой у вас объект на return стоит.

Кроме этого:

1) В этом контексте 500 Internal Server Error правильная, но не обработанная.

2) 5xx это Server Errors, а 4xx это Client Errors. В данном случае эта ошибка сервера и она правильная и ее не нужно менять на 4xx. Читайте, что такое REST и как правильно его писать.

3) Если хотите, чтобы именно в этой ситуации не было 500 тогда в объекте User поле username допишите в аннотацию аттрибут @Column(nullable=true).

4) Во избежания таких ошибок в будущем, не используйте напрямую Entity как RequestBody в контроллере. Потому, что объект выходит из под вашего контроля еще до того, как он попадает в сам метод. Для этого создавайте Proxy классы, которые в Spring Framework называются DTO(Data transfer object), а после этого допишите конвертер в котором уже будет происходить конвертация этого объекта в Entity и вся остальная валидация.

1
  • Спасибо за развернутый ответ! 23 янв 2018 в 14:32
0

Используйте bean validation

Модель:

public class User {
    @NotNull
    private String username;

    @NotNull
    private String password;

    ...
}

Контроллер:

public ResponseEntity<String> registration(@Valid @RequestBody User newUser, BindingResult bindingResult) {
    ...
}
5
  • Все равно приходит 500 17 янв 2018 в 19:15
  • Анализируйте stack trace, смотрите где именно возникает ошибка сервера и почему. 18 янв 2018 в 4:56
  • Конечно приходит 500. Пишет же черным по белому @NotNull, а приходит null.
    – calm27
    23 янв 2018 в 13:16
  • @DimaKhodan если bean не проходит валидацию, Spring возвращает bad request. Шанс получить ошибку 500 в данном случае отсутствует. 23 янв 2018 в 13:19
  • @SergeyGornostaev вы правы, но не учли нюанс спрингового exception handler, которые любую не обработанную ошибку врапает в 500ю, что и происходит в данной ситуации.
    – calm27
    23 янв 2018 в 13:22

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.