Code review Sprint 7

[BigDeepBlue]

Привет! Присоеденюсь к прошлому отзыву - работа отличная! У меня всего несколько рекомендаций:

1. Давайте сделает возможность отключать лимитер, например, в тестах или дев окружении, для этого настраивайте через конфиг параметр у Limiter - enabled
2. Аналогично с трейсером - сделайте его подключение через отдельный параметр в конфиге
3. Можно лучше: тут у вас как-то странно импорт flask отбился отдельно, он должен быть в верхней группе
4. Можно лучше: тут по идее стоит просто возвращать статус код (404) с текстом, что такого провайдера нет. Щас у вас будет как 500ка возвращаться, это не очень правильно
5. Если есть время, то рекомендую попробовать капчу или TOTP

[ReznikovRoman]

@BigDeepBlue Привет, спасибо за ревью!

1. сделаем
2. сделаем
3. Нет, это наша настройка isort’а. Отвечали на одном из прошлых ревью
   уже)
4. Именно для этого у нас настроена обработка ошибок) Вот сам обработчик, а вот экспешн. И вот я даже сейчас тест нашел, который специально проверяет этот случай
5. В этом спринте уже вряд ли успеем капчу сделать

Edit: готово - #36

[BigDeepBlue]

Именно для этого у нас настроена обработка ошибок) Вот сам обработчик, а вот экспешн. И вот я даже сейчас тест нашел, который специально проверяет этот случай

проглядел обработчик :( Хорошо)

По остальным замечаниям - ок) Принято!