Code review

[BigDeepBlue]

Здравствуйте ! Отличная работа ! Есть несколько рекомендаций:

1. Здесь предполагается что сервис будет высоконагруженным, нужно поместить его за nginx
2. Одним из заданий данного спринта было исследование хранилищ - не нашел, добавьте свое исследование и выводы в репозиторий.
3. Тут в реальных условиях не забывайте менять дефолтные пароли.
4. Тут лучше указывайте точные версии библиотек, так не сломается неожиданно, после того как авторы поменяют что нибудь )
5. Тут версия fastapi достаточно старая, можно обновить (и посмотреть остальные библиотеки). Часто более новые версии имеют более расширенный функционал и исправлены проблемы предыдущих версий.
6. Тут советовал бы использовать gunicorn для запуска https://fastapi.tiangolo.com/deployment/server-workers/#server-workers-gunicorn-with-uvicorn и вот об этом самой документации uvicorn - https://www.uvicorn.org/deployment/#gunicorn
7. Здесь не нашел где используется этот скрипт.
8. Тут лучше установку apt зависимостей перенести повыше (например сразу после FROM) - так Вы не будете ожидать их установку если что то поменяется в requirements. Еще можно удалить списки - https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#sort-multi-line-arguments так образ станет чуточку меньше.
9. Тут импорты немного перемешаны, лучше следовать https://peps.python.org/pep-0008/#imports при организации импортов.
10. Тут лишняя пустая строка - могу посоветовать https://black.readthedocs.io/en/stable/ - очень помогает избегать рутины по форматированию кода (попробуйте прогнать файлики проекта, поправит лишние пустые строки и др).
11. Здесь - зачем requests ?

[acetone415]

пункт 7 - скрипт используется здесь.

[BigDeepBlue]

Молодцы, все поправили, принято !