Ревью

[kazqvaizer]

@nkiryanov классная работа и, кажется, есть вообще все, что может пригодиться в задаче уведомлений по вебсокету.

1. Совершенно точно нужно добавить картинку-схему чтобы объяснить что происходит. Лучше две картинки, одну про связь элементов системы (цепочку из реббита, вебсокетов и клиента), и про протокол общения с авторизацией, проверкой JWT и инвалидацией сессий.

2. Предлагаю убрать вот эту штуку с получением ключей налету. Большинству людей достаточно будет добавить секретный или публичный ключ из ENV для валидации JWT. Упростим?

3. Предлагаю убрать broadcast, тут и может быть где-то еще, чтобы заменить на цикл с await connection.send(data), тк. синхронный broadcast может вести к переполнению буфера. Бродкаст лучше имплементировать, если понятны риски.

4. Кажется, что тут тоже нужно прикрутить stop_signal? Дока тут .

5. Нужно добавить Dockerfile, а также добавить инструкции к запуску кода.

6. Предлагаю переименовать WebSocketsAccessGuardian в понятное, типа SessionExpirationChecker

7. Наверное, можно доработать сервис, если добавить сюда еще и dummy producer данных, которые пришли обратно из вебсокета, но возможно, что это уже за рамками этого темплейта.

[nkiryanov]

Сделал по каждому кроме 7 сделал задачи — его в обозримом будущем не хочу делать. Только если появится необходимость (просто в моём бэклоге есть задачи интереснее 🤷‍♂️)

Остальное или сделаю сегодня или думаю что не нужно и обсуждаем в связанном issue.