django-selectel-storage
django-selectel-storage copied to clipboard
marazmiki
Добавить поддержку метаданных
Привет, Михаил! Мне нужно при выгрузке файлов добавлять метаданные (заголовки для Content-Disposition). Предлагаю сделать так, как реализовано на амазоновской либе (см. тут ) Настройка AWS_S3_METADATA. Как считаешь? Могу сделать пулл реквест
Здравствуйте!
Я так понимаю, что эти метаданные — некоторое отступление от "интерфейса" хранилища джанги?
Да. Но не совсем. Думаю, будет логично, что storage не только знает куда и как сохранить файл, но умеет при сохранении добавлять метаданные.
Может, это и логично, но уж коль скоро этой функциональности в других сторадж-бэкендов нет, это поведение нестандартное. Нужно это понимать.
С другой стороны, если они только в сохранении используются и выставляются неявно, внутри _save(), то можно и сделать, я не против :)
Согласен, поведение нестандартное. Но, если не знать об этой дополнительной возможности, она никак не повлияет на предсказуемость системы. Ну так что, мне дописать и сделать пулл реквест?
А давайте :)
Только, пожалуйста, с тестами и кодом по PEP-0008, ок? Трэвис, если что, подскажет