Roman

+9
с 2023
0 подписчиков
0 подписок

Лиду и тестировщику тоже самое скажешь? Не долго проработаешь на бэке, сударь.

1
Ответить

Название файла парси из ссылки, принты на логгинг меняй. И логи на английском. Вынеси логику и клиентский код отдельно. Не вижу обработки ошибок для запросов. Библиотеку поменяй на httpx. Почему не class based, переписывай и сделай абстракцию для http клиента от которой и наследуйся. Если отдельный файл со скриптом то где if __name__ ... ? Если запросов таких много будет, юзай асинхронку сразу лучше.

Так в целом по верхам прошлись, тегнешь после некст коммита, пока денаю пулл реквест.

P.s. Добавь ещё pillow, чекай что картинка валидная, пережми в jpeg и задай Макс разрешение по ширине и высоте чтобы большие картинки не сохранять в оригинале. Место на сторедже беречь надо, а не закидывать картинки по 50Мб.

8
Ответить