Size: a a a

NestJS — русскоязычное сообщество

2020 August 11

JB

Joe Beretta in NestJS — русскоязычное сообщество
Maksim Zakharov
если подумаешь что это typescript - не удивляйся)
Ну я б не удивлялся, зная, что создали что дотнэт, что тс мелгомягкие) 😅
источник

SU

Sergey Ududenko in NestJS — русскоязычное сообщество
Привет всем, подскажите как получить обьект реквеста в кастомном декораторе на хендлере
источник

A

Artem in NestJS — русскоязычное сообщество
Sergey Ududenko
Привет всем, подскажите как получить обьект реквеста в кастомном декораторе на хендлере
источник

A

Artem in NestJS — русскоязычное сообщество
нужную часть сам достанешь)
источник

A

Artem in NestJS — русскоязычное сообщество
а вообще зачем кастомный декоратор, если в несте есть встроенный?
источник

SU

Sergey Ududenko in NestJS — русскоязычное сообщество
@kjljxybr, спасибо, особенности проекта)
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
Oleg Rizhkov
removeTaskById(id: string): void {
 this.getTaskByIdOrThrowNotFound(id)
 this.tasks = this.tasks.filter((task) => task.id !== id)
}
А зачем лишний раз проверять? Удаляй и все, если есть - удалится, если нет то нет
источник

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество
🏡 ILshat Khamitov
А зачем лишний раз проверять? Удаляй и все, если есть - удалится, если нет то нет
тоже верно. но я скорее про абстрактный пример, если бы там был не remove.
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
Oleg Rizhkov
тоже верно. но я скорее про абстрактный пример, если бы там был не remove.
Давай другой пример)
источник

VA

Veaceslav Artiom in NestJS — русскоязычное сообщество
Ребят а как указать что field может не быть в Body, но если будет тогда что бы не был пустым ?
источник

LK

L K in NestJS — русскоязычное сообщество
Oleg Rizhkov
removeTaskById(id: string): void {
 this.getTaskByIdOrThrowNotFound(id)
 this.tasks = this.tasks.filter((task) => task.id !== id)
}
я когда-то был за такие подходы
но потом понял что это не совсем с точки зрения бизнеса правильно

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

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество
L K
я когда-то был за такие подходы
но потом понял что это не совсем с точки зрения бизнеса правильно

потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
наружу - куда? это сервис.
источник

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество
L K
я когда-то был за такие подходы
но потом понял что это не совсем с точки зрения бизнеса правильно

потому что чувак который захочет удалить записи либо чего может перебором узнать существует ли что-то еще
ну типа перебор id
это не то что бы дыра в безопасности, но ты открываешь "нутро" бекенда наружу
то есть, если мне не нужно возвращать 404, если чувак хочет что-то сделать с объектом, которого нет?
источник

LK

L K in NestJS — русскоязычное сообщество
Oleg Rizhkov
то есть, если мне не нужно возвращать 404, если чувак хочет что-то сделать с объектом, которого нет?
это скорее не для текущего юзера так надо делать который и так умеет право удалять, а для левого

типа при попытки удаления левым юзером каких-то данных разумнее кинуть 404, чем 403 что ты не можешь удалить
так ты левому юзеру не говоришь о том, есть ли такая запись или нет и может ли он ее удалить
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
ну а зачем 404 даже так кидать, он же каким то образом получил список и список уже не актуальный, вот актуальность нужно как то поддерживать и он не сможет удалить того чего нет
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
выкидыванием 404 ты просто приносишь ненужную логику проверки в метод усложняя код и не решаешь истинную проблему
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
еще и базу грузишь лишним запросом
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
ну мое мнение, хз, твой код тебе решать
источник

LK

L K in NestJS — русскоязычное сообщество
🏡 ILshat Khamitov
ну а зачем 404 даже так кидать, он же каким то образом получил список и список уже не актуальный, вот актуальность нужно как то поддерживать и он не сможет удалить того чего нет
это какая-то другая задача )
источник

🏡K

🏡 ILshat Khamitov in NestJS — русскоязычное сообщество
)
источник