JB
если подумаешь что это typescript - не удивляйся)
Ну я б не удивлялся, зная, что создали что дотнэт, что тс мелгомягкие) 😅
Size: a a a
2020 August 11
SU
Привет всем, подскажите как получить обьект реквеста в кастомном декораторе на хендлере
A
Привет всем, подскажите как получить обьект реквеста в кастомном декораторе на хендлере
A
нужную часть сам достанешь)
A
а вообще зачем кастомный декоратор, если в несте есть встроенный?
SU
@kjljxybr, спасибо, особенности проекта)
🏡K
removeTaskById(id: string): void {
this.getTaskByIdOrThrowNotFound(id)
this.tasks = this.tasks.filter((task) => task.id !== id)
}А зачем лишний раз проверять? Удаляй и все, если есть - удалится, если нет то нет
OR
А зачем лишний раз проверять? Удаляй и все, если есть - удалится, если нет то нет
тоже верно. но я скорее про абстрактный пример, если бы там был не remove.
🏡K
тоже верно. но я скорее про абстрактный пример, если бы там был не remove.
Давай другой пример)
VA
Ребят а как указать что field может не быть в Body, но если будет тогда что бы не был пустым ?
LK
removeTaskById(id: string): void {
this.getTaskByIdOrThrowNotFound(id)
this.tasks = this.tasks.filter((task) => task.id !== id)
}я когда-то был за такие подходы
но потом понял что это не совсем с точки зрения бизнеса правильно
потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
но потом понял что это не совсем с точки зрения бизнеса правильно
потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
OR
я когда-то был за такие подходы
но потом понял что это не совсем с точки зрения бизнеса правильно
потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
но потом понял что это не совсем с точки зрения бизнеса правильно
потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
наружу - куда? это сервис.
OR
я когда-то был за такие подходы
но потом понял что это не совсем с точки зрения бизнеса правильно
потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
но потом понял что это не совсем с точки зрения бизнеса правильно
потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
то есть, если мне не нужно возвращать 404, если чувак хочет что-то сделать с объектом, которого нет?
LK
то есть, если мне не нужно возвращать 404, если чувак хочет что-то сделать с объектом, которого нет?
это скорее не для текущего юзера так надо делать который и так умеет право удалять, а для левого
типа при попытки удаления левым юзером каких-то данных разумнее кинуть 404, чем 403 что ты не можешь удалить
так ты левому юзеру не говоришь о том, есть ли такая запись или нет и может ли он ее удалить
типа при попытки удаления левым юзером каких-то данных разумнее кинуть 404, чем 403 что ты не можешь удалить
так ты левому юзеру не говоришь о том, есть ли такая запись или нет и может ли он ее удалить
🏡K
ну а зачем 404 даже так кидать, он же каким то образом получил список и список уже не актуальный, вот актуальность нужно как то поддерживать и он не сможет удалить того чего нет
🏡K
выкидыванием 404 ты просто приносишь ненужную логику проверки в метод усложняя код и не решаешь истинную проблему
🏡K
еще и базу грузишь лишним запросом
🏡K
ну мое мнение, хз, твой код тебе решать
LK
ну а зачем 404 даже так кидать, он же каким то образом получил список и список уже не актуальный, вот актуальность нужно как то поддерживать и он не сможет удалить того чего нет
это какая-то другая задача )