Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code revierw #1

Open
BigDeepBlue opened this issue Jul 7, 2022 · 1 comment
Open

Code revierw #1

BigDeepBlue opened this issue Jul 7, 2022 · 1 comment

Comments

@BigDeepBlue
Copy link
Collaborator

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

  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
Copy link
Collaborator Author

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant