Сохранение изображений и эскизов на s3 в django

Ответ 1

Как автор миниатюры sorl, я действительно заинтересован в решении этого вопроса, если он не работает так, как я предполагал. Если значение ключа sotre заполнено, оно будет хранить: имя, хранилище и размер. Я сделал предположение, что URL-адрес основан на имени и, следовательно, не должен вызывать вызовы памяти. Глядя на хранилища django, https://github.com/e-loue/django-storages/blob/master/storages/backends/s3boto.py#L214, кажется, что это безопасное предположение. По какой-то причине в вашем патче вы исправили метод чтения. При создании эскиза экземпляр ImageFile извлекается из кеша (если его не создавать), вы можете, конечно, вызывать чтение, которое будет читать файл, но предполагаемое использование -.url, которое вызывает URL-адрес на хранилище с кешированным именем, которое должно быть оператором доступа без хранения. Не могли бы вы попытаться изолировать свою проблему до места, где в вашем коде происходит доступ к хранилищу?

Также убедитесь, что у вас включен THUMBNAIL_DEBUG, и что у вас правильно сохранено хранилище ключей.

Ответ 2

Я не уверен, что у вас проблема такая же, как у меня, но я обнаружил, что доступ к свойству width или height обычного Django ImageField будет считывать файл с базы данных хранилища, загружать его в PIL и возвращать размеры оттуда. Это особенно дорого стоит с удаленным бэкэнд, как мы используем, и у нас есть очень тяжелые страницы.

https://code.djangoproject.com/ticket/8307 был открыт для решения этой проблемы, но разработчики Django закрыты как wontfix, потому что они хотят, чтобы свойства width и height всегда возвращали true значения. Поэтому я просто monkeypatch _get_image_dimensions(), чтобы использовать эти поля, что предотвращает большое количество сообщений boto и улучшает время загрузки страницы.

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

from django.core.files.images import ImageFile, get_image_dimensions
def _get_image_dimensions(self):
    from numbers import Number
    if not hasattr(self, '_dimensions_cache'):
        close = self.closed
        if self.field.width_field and self.field.height_field:
            width = getattr(self.instance, self.field.width_field)
            height = getattr(self.instance, self.field.height_field)
            #check if the fields have proper values
            if isinstance(width, Number) and isinstance(height, Number):
                self._dimensions_cache = (width, height)
            else:
                self.open()
                self._dimensions_cache = get_image_dimensions(self, close=close)
        else:
            self.open()
            self._dimensions_cache = get_image_dimensions(self, close=close)

    return self._dimensions_cache
ImageFile._get_image_dimensions = _get_image_dimensions

Ответ 3

Посмотрев на билет @shadfc django, я повторно выполнил команду monkeypatch следующим образом:

from django.core.files.images import ImageFile
def _get_image_dimensions(self):
    if not hasattr(self, '_dimensions_cache'):
        if getattr(self.storage, 'IGNORE_IMAGE_DIMENSIONS', False):
            self._dimensions_cache = (0, 0)
        else:
            close = self.closed
            self.open()
            self._dimensions_cache = get_image_dimensions(self, close=close)
    return self._dimensions_cache
ImageFile._get_image_dimensions = _get_image_dimensions

Чтобы использовать его, просто добавьте IGNORE_IMAGE_DIMENSIONS = True в свой класс хранения, и он не будет затронут, чтобы получить размеры изображения. Скорее всего:

from storages.backends.s3boto import S3BotoStorage
S3BotoStorage.IGNORE_IMAGE_DIMENSIONS = True

Мне все еще нужно исследовать, где используются числа, чтобы знать, может ли простое возвращение (0, 0) привести к какой-либо проблеме, но пока что ошибка не возникла.