search

TRUEVORO / ugc_sprint_2

Спринт 9. Финальный спринт командного проекта по UGC
0 stars 0 forks source link

Code review #4

Open BigDeepBlue opened 1 year ago

BigDeepBlue commented 1 year ago

Отлично разобрались с конфигом elk/logstash/logstash.conf 👍 Вот, вдруг будет интересно, он еще и сообщение в Telegram может отправить при неоходимости:

if какое-нибудь-условие {
    http { 
        url => "https://api.telegram.org/bot${bot_key}/sendMessage" 
        http_method => "post" 
        format => message 
        content_type => "application/json" 
        message => '{"chat_id":"${chat_id}","text":"Have error in log"}' 
    } 
}

  1. Поискал по README и не нашел, вот этого пункта задания:

    Результаты тестирования скорости работы с хранилищем в формате .md файла (README.md).

  2. Кроме лайков в задании было про закладки и рецензии.

  3. В прошлых спринтах реализовывали сервис авторизации. Его нужно использовать, вот тут и далее вы принимаете user_id пользователя в качестве параметра. Т.е. кто угодно может поставить лайк за другого пользователя. Это не очень правильно. Тут 2 варианта:

    • либо тут же в сервисе разбирайте JWT и извлекайте id - при таком подходе мы будем зависить от времени жизни токена, т.е. пользователь уже отключен, но до конца жизни токена будет пользоваться сервисом
    • либо сделайте middleware и переправляйте JWT в сервис авторизации и получайте назад все данные пользователя.
BigDeepBlue commented 1 year ago

LGTM