search

SeergeyL / Auth-sprint-2

0 stars 0 forks source link

Code revierw #1

Open BigDeepBlue opened 2 years ago

BigDeepBlue commented 2 years ago

Хорошая работа, но все же:

  1. Попробуйте https://pydantic-docs.helpmanual.io/usage/settings/ , станет очевидным насколько удобнее и лаконичнее будет работа с настройками, опять же валидация, что тоже очень хорошо.
  2. Попробуйте добавить еще одного провайдера авторизации, будет очень много общего с классом YandexOAuth, и тогда это общее можно будет вынести в базовый класс BaseOAuth, сделав его немного "менее" абстрактным.

  3. Не нашел реализацию задачи:

    Сейчас сервис Auth беззащитен перед DDOS-атаками. Чтобы сберечь важный элемент системы от чрезмерной нагрузки, которая может вывести его из строя, реализуйте алгоритм Leaky bucket или Token bucket, используя Redis для синхронизации реплик.

    Можно и https://flask-limiter.readthedocs.io/en/stable/ использовать, как вариант.

  4. В сервис выдачи контента не пускает, а там должно быть:

    Создайте интеграцию Auth-сервиса и AsyncAPI-сервиса, используя контракт, который вы сделали в прошлом задании.

BigDeepBlue commented 2 years ago

LGTM