Вопрос общего характера по бекенду: должен ли я валидировать и

Question

Вопрос общего характера по бекенду: должен ли я валидировать и

защищать внутреннюю реализацию (сервисы и сущности) или только внешний интерфейс (контроллеры и дто?)

Представим такую ситуацию, у меня на проекте есть Entity и Dto. Первый я не валидирую никак, просто задаю тип данных - строка. Второй я очень жестко валидирую - принимать только строку с хешем алгоритма sha3-256

Представим другую ситуацию, у меня на проекте есть Service и Controller для ресайза изображений. Первый я не валидирую никак, он может, скажем, ресайзить картинку хоть до 400000000x400000000 (что будет крайне долго). Второй я жестко валидирую - если переданная пользователем ширина или высота больше 2000, то я троваю ошибку BadRequest.

Это ОК или нужно валидировать данные на уровне внутренней реализации, а не внешнего интерфейса? (тровать ошибки в сервисе, делать валидацию Entity при записи в БД [при том, что по типам БД оно проходит] и т.д)

#backend #javascript #node.js #programming #russian

0

07.05.2021

13 ответов

33 просмотра

✙ Nikita 🇺🇦🇵🇱🇱🇻🇱🇹🇪🇪🇺🇸🇬🇧🇨🇿🇫🇷🇩🇪🇮🇹🇧🇬🇹🇼🇯🇵🇰🇷🇹🇷🇪🇸🇬🇷

по максимуму валидации на входных точках (контроллеры) и в самой бд (констрейнты)

0

07.05.2021

Veli Автор вопроса

не совсем понял, как будто бы второе предложение противоречит первому) Так Entity должна защищаться либо защита на уровне контроллера - это ок?

0

07.05.2021

Veli Автор вопроса

так вот проблема, что сама БД не имеет валидация типа «хочу хэш вот в этой вот колонке, а не просто строчку». Могу на уровне сервиса или репозитория это вручную предотвращать, но стоит ли?

0

07.05.2021

✙ Nikita 🇺🇦🇵🇱🇱🇻🇱🇹🇪🇪🇺🇸🇬🇧🇨🇿🇫🇷🇩🇪🇮🇹🇧🇬🇹🇼🇯🇵🇰🇷🇹🇷🇪🇸🇬🇷

имею ввиду что вот например в запросе POST /api/orders/status в контроллере статус "sent" - это валидное значение в теле запроса но сам энтити не может в нем оказаться, поскольку например заказ уже закрыт

0

07.05.2021

Vladislav Stepanov

простые ошибки валидации(тип, формат не тот) валидируешь в контроллере Все что сложнее(уникальность) валидируешь в сервисе Если я правильно понял вопрос

0

07.05.2021

Veli Автор вопроса

уникальность бд сама заблочит, так что тут можно даже не думать) Значит формат все-таки к контроллеру, я тоже к этому склоняюсь

0

07.05.2021

Veli Автор вопроса

хм, тоже интересный пример

0

07.05.2021

Andy Cox

имхо всё постороннее должно отваливаться еще на этапе middleware

0

07.05.2021

✙ Nikita 🇺🇦🇵🇱🇱🇻🇱🇹🇪🇪🇺🇸🇬🇧🇨🇿🇫🇷🇩🇪🇮🇹🇧🇬🇹🇼🇯🇵🇰🇷🇹🇷🇪🇸🇬🇷

если это то что-то, что должно отвалиться, требует проверки в бд или еще где то, это тоже мидлвара?

0

07.05.2021

Andy Cox

размер картинки?

0

07.05.2021

✙ Nikita 🇺🇦🇵🇱🇱🇻🇱🇹🇪🇪🇺🇸🇬🇧🇨🇿🇫🇷🇩🇪🇮🇹🇧🇬🇹🇼🇯🇵🇰🇷🇹🇷🇪🇸🇬🇷

это общий вопрос был

0

07.05.2021

Veli Автор вопроса

это не размер картинки, это параметры, по которым я ресайзю картинку, если речь конкретно про мое уточнение)

0

07.05.2021

Accepted Answer

✙ Nikita 🇺🇦🇵🇱🇱🇻🇱🇹🇪🇪🇺🇸🇬🇧🇨🇿🇫🇷🇩🇪🇮🇹🇧🇬🇹🇼🇯🇵🇰🇷🇹🇷🇪🇸🇬🇷

Entity должна делать все чтобы она не могла оказаться в невалидном состоянии относительно бизнес логики по идеи не нужно, поскольку это только больше в кашу прерварит твой код бизнес логики, смешав его с кодом проверки данных

0

07.05.2021

170 похожих чатов

Вопрос общего характера по бекенду: должен ли я валидировать и

13 ответов

Похожие вопросы