andisaev / auth_sprint_2 Goto Github PK

Отлично. Рекорд по реализованному количеству способов авторизации :)

1. Нужно было реализовать задачу: Создайте интеграцию Auth-сервиса и AsyncAPI-сервиса, используя контракт, который вы сделали в прошлом задании. Т.е. добавить middleware, например, которая будет разбирать токен или ходить с ним в сервис авторизации и в зависимости от прав давать тот или иной контент. Например, авторизованным - все, гостям - только списком фильмы без подробностей. Не обязательно это делать тут, вполне сойдет репо https://github.com/AndIsaev/Async_API_sprint_2
2. Зря вы роуты этого спринта реализовали в app.py. Там же код может расти с развитием проекта. Лучше сразу положиться на blueprint
3. def google(provider: str), кажется есть смысл переименовать, метод же у вас универсальный, не только для google.
4. Импорты проекта рекомендуется разделять на три группы символом новой строки. Группы такие: импорты стандартных библиотек, импорты установленных пакетов, импорты твоего проекта.
   Так же, чтобы не делать такие вещи каждый раз в ручную, рекомендую добавить в pre-commit хук библиотеку isort (https://pycqa.github.io/isort/).
5. Имена провайдеров (тут и тут и далее) лучше поместить в enum. Работа с литералами - всегда не очень хорошая практика. Есть вероятность, что с развитием проекта их придется еще использовать в других местах кода.