Code review

[BigDeepBlue]

Привет! Хорошая работа! Здорово, что вы сделали дополнительное задание с двухфакторкой. Ниже список небольших замечаний:

1. Совет: при генерации миграций рекомендую передавать параметр с кастомным именем, чтобы в таком наборе цифр было проще разбираться
2. Можно лучше: заводить бд в редис на каждый чих выглядит довольно затратно + держите больше открытых коннектов. Обычно делят на бд по критичности данных: например, важные токены и данные в одну бд, а в другую бд - кеш. А данные внутри одной бд можно разруливать префиксами вроде ref-token, revoked-token и т.д.
3. Почему сеттинги redis здесь, а не в конфиге?
4. По одному из заданий было необходимо настроить партицирование таблицы юзеров. Это обязательное задание, можете партицировать не юзеров, а историю логинов, например, по типу в user-agent: desktop или mobile
5. По умолчанию requests ждет ответ от сервера бесконечно долго, было бы неплохо в конфиг вынести максимальный таймаут, который вы готовы ждать
6. Можно лучше: id тоже не помешает вынести в конфиг
7. Давайте подключение трейсера занесем под if, чтобы его можно было отключать по необходимости, например, во время запуска тестов
8. Давайте вместо числовых статус-кодов использовать http.HTTPStatus
9. Тут видимо не поправили на сваггер для яндекса
10. Интересная конечно идея ограничивать только авторизованных юзеров, но тот же ddos идет обычно с неавторизованных адресов. Поэтому давайте вешать лимит не на токен, а на конкретное устройство в сети, как идентифицировать - выбирайте сами. Можете поглядеть под капот https://flask-limiter.readthedocs.io/en/stable/

[BigDeepBlue]

Небольшой коммент тут:
#116 (comment)

Принято!