vladefr97 / auth_sprint_2 Goto Github PK

Привет! Хорошая работа. У меня всего несколько замечаний:

1. Совет: могу предложить вам использовать make, он позволяет существенно сократить стандартные команды docker-compose: https://habr.com/ru/company/ruvds/blog/486682/
2. Тут странно используете синтаксис регулярок, а еще лучше было бы ограничить маршруты до location /api, чтобы не обрабатывать лишние запросы вашим приложением
3. Можно лучше: делать несколько объектов с настройками не очень правильно. Рекомендую группировать все настройки в одном классе, тут пример: https://pydantic-docs.helpmanual.io/usage/settings/
4. Давайте сделаем отдельный параметр в конфиге, по которому будем отключать трейсер в тех же тестах. Так же по 21 строке - логи в консоль стоит выводить только по config.debug=true, в проде будете только засорять консоль
5. У вас ключ привязан к минуте, а дальше вы строите свою логику с интервалами, которая не будет работать из-за формата ключа, рекомендую дорефачить это, либо посмореть как реализовано в https://flask-limiter.readthedocs.io/en/stable/
6. Можно лучше: я бы вам рекомендовал разнести данные кеша для лимитера и данные jwt-токенов. Это можно сделать здесь. Если создать два инстанса с разными db id
7. А если здесь придет имя клиента, которого нет в словаре?)
8. С версии python 3.9 для стандартных коллекций больше не нужен модуль typing: https://docs.python.org/3.9/whatsnew/3.9.html#type-hinting-generics-in-standard-collections. Поправьте во всем проекте
9. Отсюда и ниже вы будете дублировать в каждом провайдере, что приведет к проблемам с изменением кода в последующем. Давайте вынесем либо в базовый класс, либо в функцию хелпер. Так же подумайте над тем, что у вас примерно одинаковая логика регистрации должать быть как через oauth, так и через стандартную регистрацию, возможно придумаете как сделать общий хелпер. Но план минимум - вынести код регистрации из конкретного провайдера