Добавить поддержку метаданных

[davitv]

Привет, Михаил! Мне нужно при выгрузке файлов добавлять метаданные (заголовки для Content-Disposition).
Предлагаю сделать так, как реализовано на амазоновской либе (см. тут )
Настройка AWS_S3_METADATA.
Как считаешь? Могу сделать пулл реквест

[marazmiki]

Здравствуйте!

Я так понимаю, что эти метаданные — некоторое отступление от "интерфейса" хранилища джанги?

[davitv]

Да. Но не совсем. Думаю, будет логично, что storage не только знает куда и как сохранить файл, но умеет при сохранении добавлять метаданные.

[marazmiki]

Может, это и логично, но уж коль скоро этой функциональности в других сторадж-бэкендов нет, это поведение нестандартное. Нужно это понимать.

С другой стороны, если они только в сохранении используются и выставляются неявно, внутри _save(), то можно и сделать, я не против :)

[davitv]

Согласен, поведение нестандартное. Но, если не знать об этой дополнительной возможности, она никак не повлияет на предсказуемость системы.
Ну так что, мне дописать и сделать пулл реквест?

[marazmiki]

А давайте :)

Только, пожалуйста, с тестами и кодом по PEP-0008, ок? Трэвис, если что, подскажет

[davitv]

Ок. Спасибо!