search

it-beard / bloggers-cms

Content-management system for bloggers
https://github.com/it-beard/bloggers-cms/tree/develop/docs
Apache License 2.0
51 stars 14 forks source link

Логирование исключений #79

Closed Alex2021cyberhuman closed 3 years ago

Alex2021cyberhuman commented 3 years ago

Добавить системы диагностики и логирования WebApi

Alex2021cyberhuman commented 3 years ago

Готов решить этот вопрос с помощью нлог и фильтров обработки исключений

itbeard commented 3 years ago

Готов решить этот вопрос с помощью нлог и фильтров обработки исключений

в добрый путь

Alex2021cyberhuman commented 3 years ago

Сейчас, если пользователь api запросит не существующею сущность он получит ошибку сервера т.к. методы репозиториев возвращают null если сущность не найдена, а mapper пытается преобразовать null в специальный тип response. Я предлагаю выкидывать тип исключений NotFoundException и преобразовывать в спец. тип ответа со статус кодом 404 с помощью фильтров действий. Так же мне надо переделать предыдущую систему обработки исключений в контроллерах.

pingvin1308 commented 3 years ago

Учитывая что проект впитывает в себя все модные подходы. То апи следует делать таким, чтобы оно возвращало ошибку с ProblemDetails. Ну и да, при не нахождении сущности можно вернуть 404-ую

Alex2021cyberhuman commented 3 years ago

Что должно содержать problem details?

pingvin1308 commented 3 years ago

в asp.net core есть класс для этого. Это уже своего рода стандарт для апи ошибок. https://tools.ietf.org/html/rfc7807

itbeard commented 3 years ago

SerialLog https://serilog.net

itbeard commented 3 years ago

пока не актуально