Azure Blob Storage Plugin

sam · 07.Март.2019 21:00:15

Awesome, @vinothkannans can you review and merge ?

AJRT · 23.Июль.2019 15:33:44

Привет, @maja

У меня возникла проблема, которая может быть связана с этим плагином.
Дополнительная информация об этой проблеме:

Не могли бы вы проверить, является ли проблема в этом плагине, и попытаться её исправить?

И, возможно, это тоже связано:

schungx · 29.Август.2019 06:29:27

Любой файл, загруженный через short-url, автоматически получает ошибку 500.

В логах указано следующее:

NoMethodError (undefined method `url_for' for #<FileStore::AzureStore:0x00007f4553fc2ae8>)
/var/www/discourse/app/controllers/uploads_controller.rb:107:in `show_short'

/var/www/discourse/app/controllers/uploads_controller.rb:107:in `show_short'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/base.rb:194:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/rendering.rb:30:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/callbacks.rb:42:in `block in process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:132:in `run_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/callbacks.rb:41:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/rescue.rb:22:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/instrumentation.rb:34:in `block in process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/notifications.rb:168:in `block in instrument'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/notifications/instrumenter.rb:23:in `instrument'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/notifications.rb:168:in `instrument'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/instrumentation.rb:32:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal/params_wrapper.rb:256:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-5.2.3/lib/active_record/railties/controller_runtime.rb:24:in `process_action'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/abstract_controller/base.rb:134:in `process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionview-5.2.3/lib/action_view/rendering.rb:32:in `process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.0.2/lib/mini_profiler/profiling_methods.rb:78:in `block in profile_method'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal.rb:191:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_controller/metal.rb:252:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/routing/route_set.rb:52:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/routing/route_set.rb:34:in `serve'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/journey/router.rb:52:in `block in serve'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/journey/router.rb:35:in `each'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/journey/router.rb:35:in `serve'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/routing/route_set.rb:840:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-protection-2.0.5/lib/rack/protection/frame_options.rb:31:in `call'
/var/www/discourse/lib/middleware/omniauth_bypass_middleware.rb:68:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/tempfile_reaper.rb:15:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/conditional_get.rb:25:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/head.rb:12:in `call'
/var/www/discourse/lib/content_security_policy/middleware.rb:12:in `call'
/var/www/discourse/lib/middleware/anonymous_cache.rb:220:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/session/abstract/id.rb:232:in `context'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/session/abstract/id.rb:226:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/cookies.rb:670:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/callbacks.rb:28:in `block in call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activesupport-5.2.3/lib/active_support/callbacks.rb:98:in `run_callbacks'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/callbacks.rb:26:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/debug_exceptions.rb:61:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/show_exceptions.rb:33:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/logster-2.3.2/lib/logster/middleware/reporter.rb:43:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/rack/logger.rb:38:in `call_app'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/rack/logger.rb:28:in `call'
/var/www/discourse/config/initializers/100-quiet_logger.rb:18:in `call'
/var/www/discourse/config/initializers/100-silence_logger.rb:31:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/remote_ip.rb:81:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/request_id.rb:27:in `call'
/var/www/discourse/lib/middleware/enforce_hostname.rb:17:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/method_override.rb:22:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/executor.rb:14:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/sendfile.rb:111:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-mini-profiler-1.0.2/lib/mini_profiler/profiler.rb:171:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/message_bus-2.2.2/lib/message_bus/rack/middleware.rb:57:in `call'
/var/www/discourse/lib/middleware/request_tracker.rb:169:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/engine.rb:524:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/railtie.rb:190:in `public_send'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/railties-5.2.3/lib/rails/railtie.rb:190:in `method_missing'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/urlmap.rb:68:in `block in call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/urlmap.rb:53:in `each'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rack-2.0.7/lib/rack/urlmap.rb:53:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:605:in `process_client'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:700:in `worker_loop'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:548:in `spawn_missing_workers'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/lib/unicorn/http_server.rb:144:in `start'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/unicorn-5.5.1/bin/unicorn:128:in `<top (required)>'
/var/www/discourse/vendor/bundle/ruby/2.6.0/bin/unicorn:23:in `load'
/var/www/discourse/vendor/bundle/ruby/2.6.0/bin/unicorn:23:in `<main>'

Я почти уверен, что в файле https://github.com/discourse/discourse-azure-blob-storage/blob/master/lib/azure_blob_store.rb отсутствует реализация метода url_for…

gerhard · 29.Август.2019 06:58:30

У нас был PR по этой теме, который, к сожалению, не был принят, но там есть отличные комментарии на случай, если кто-то захочет заняться этой задачей.

github.com/discourse/discourse-azure-blob-storage

FIX: Implement url_for (#11)

master ← localjo:implement-url-for

closed 03:36PM - 08 Aug 19 UTC

localjo

+19 -0

This PR implements [the `url_for` function](https://github.com/discourse/discour…se/pull/7845) which Discourse uses to determine the URL of a download, taking into account the `force_download` parameter to [correctly set the `content-disposition` parameter for Azure](https://docs.microsoft.com/en-us/rest/api/storageservices/constructing-a-service-sas#specifying-query-parameters-to-override-response-headers-blob-service-and-file-service-only). This fixes [the bug that was preventing users from downloading files stored in Azure](https://meta.discourse.org/t/user-archive-impossible-to-download-error-nomethoderror-filestore-azurestore/123687).

Как я уже отмечал в FIX: Implement url_for by localjo · Pull Request #11 · discourse/discourse-azure-blob-storage · GitHub, если не требуется поддержка приватных загрузок, то трёх новых строк из этого PR должно быть достаточно для решения проблемы.

schungx · 29.Август.2019 07:39:10

Итак… э-э… как мне тогда получить эту версию через PR?

Или, может быть, как узнать реальный URL из short-url? Есть ли где-то таблица сопоставления?

Потому что сейчас нельзя скачать ни одно вложение, и это очень раздражает…

schungx · 29.Август.2019 08:51:57

OK, команда

git clone --single-branch --branch implement-url-for https://github.com/localjo/discourse-azure-blob-storage

сейчас работает нормально. Отпишусь после дополнительных тестов.

schungx · 29.Август.2019 09:30:35

Это происходит уже несколько дней, и я считаю, что это может быть связано с этой проблемой:

/var/www/discourse/lib/file_store/base_store.rb:148:in `not_implemented'
/var/www/discourse/lib/file_store/base_store.rb:42:in `download_url'
/var/www/discourse/lib/cooked_post_processor.rb:386:in `add_lightbox!'
/var/www/discourse/lib/cooked_post_processor.rb:350:in `convert_to_link!'
/var/www/discourse/lib/cooked_post_processor.rb:682:in `block in post_process_images'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/nokogiri-1.10.4/lib/nokogiri/xml/node_set.rb:238:in `block in each'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/nokogiri-1.10.4/lib/nokogiri/xml/node_set.rb:237:in `upto'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/nokogiri-1.10.4/lib/nokogiri/xml/node_set.rb:237:in `each'
/var/www/discourse/lib/cooked_post_processor.rb:679:in `post_process_images'
/var/www/discourse/lib/cooked_post_processor.rb:42:in `block in post_process'
/var/www/discourse/lib/distributed_mutex.rb:31:in `block in synchronize'
/var/www/discourse/lib/distributed_mutex.rb:27:in `synchronize'
/var/www/discourse/lib/distributed_mutex.rb:27:in `synchronize'
/var/www/discourse/lib/distributed_mutex.rb:12:in `synchronize'
/var/www/discourse/lib/cooked_post_processor.rb:38:in `post_process'
/var/www/discourse/app/jobs/regular/process_post.rb:26:in `execute'
/var/www/discourse/app/jobs/base.rb:232:in `block (2 levels) in perform'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rails_multisite-2.0.7/lib/rails_multisite/connection_management.rb:63:in `with_connection'
/var/www/discourse/app/jobs/base.rb:221:in `block in perform'
/var/www/discourse/app/jobs/base.rb:217:in `each'
/var/www/discourse/app/jobs/base.rb:217:in `perform'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:192:in `execute_job'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:165:in `block (2 levels) in process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/middleware/chain.rb:128:in `block in invoke'
/var/www/discourse/lib/sidekiq/pausable.rb:138:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/middleware/chain.rb:130:in `block in invoke'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/middleware/chain.rb:133:in `invoke'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:164:in `block in process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:137:in `block (6 levels) in dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/job_retry.rb:109:in `local'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:136:in `block (5 levels) in dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq.rb:37:in `block in <module:Sidekiq>'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:132:in `block (4 levels) in dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:250:in `stats'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:127:in `block (3 levels) in dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/job_logger.rb:8:in `call'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:126:in `block (2 levels) in dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/job_retry.rb:74:in `global'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:125:in `block in dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/logging.rb:48:in `with_context'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/logging.rb:42:in `with_job_hash_context'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:124:in `dispatch'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:163:in `process'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:83:in `process_one'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/processor.rb:71:in `run'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/util.rb:16:in `watchdog'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/sidekiq-5.2.7/lib/sidekiq/util.rb:25:in `block in safe_thread'

schungx · 29.Август.2019 09:35:40

Функция download_url должна быть другой функцией, не реализованной плагином Azure Blob.

РЕДАКТИРОВАНИЕ:

Хорошо, последствия отсутствия download_url начинают проявляться.

Работа по постобработке изображений постоянно завершается с ошибкой
Изображения не оптимизируются
Изображения не открываются в лайтбоксе
Предварительный просмотр списка тем не может получить превью изображения

Может ли кто-нибудь добрый (@gerhard?) реализовать download_url?

РЕДАКТИРОВАНИЕ 2:

@gerhard, разве это не так просто, как добавить:

def download_url(upload)
      return unless upload
      "#{upload.short_path}"
    end

Я бы хотел склонировать репозиторий, чтобы протестировать это, но Discourse сейчас стал таким огромным монстром, что я не уверен, не сломается ли что-нибудь!

РЕДАКТИРОВАНИЕ 3:

Я отправил следующий PR в localjo/discourse-azure-blob-storage:

https://github.com/localjo/discourse-azure-blob-storage/pull/1

Похоже, это решает большинство видимых проблем без каких-либо видимых негативных последствий…

rainmanjam · 23.Декабрь.2019 18:59:18

Возможно ли внести некоторые изменения, чтобы Discourse мог использовать хранилище Azure Blob для резервных копий?

NotAnonymous · 09.Январь.2020 21:55:24

Я сталкиваюсь с той же ошибкой. Был ли ваш PR принят? Если нет, можете ли вы поделиться внесёнными изменениями, чтобы я мог проверить, можно ли внедрить их на нашем сайте? У меня нет доступа к указанной ссылке на GitHub.

Спасибо!

schungx · 31.Январь.2020 09:41:00

Похоже, исходный репозиторий больше не существует.

Вы можете попробовать выполнить клонирование отсюда:

git clone --single-branch --branch implement-url-for https://github.com/schungx/discourse-azure-blob-storage.git

Здесь содержатся мои изменения. Однако, полагаю, что что-то ещё сломалось, и я ещё не исправил это.

Мой PR в официальный репозиторий:

schungx · 31.Январь.2020 11:25:33

Похоже, что в недавно произошли масштабные изменения в движке хранения S3 (который azure-blob-storage заимствует), направленные на поддержку защищённых загрузок.

В результате нарушилась обработка некоторых постов с изображениями. Генерируется такой HTML:

<img
    src="https://xxx.com/t/topic/3493"
    srcset=",  1.5x,  2x"
    style="position: absolute; top: 0px; left: 0px; width: 678px; height: 388px;" 
    class="d-lazyload d-lazyload-hidden"
>

Обратите внимание на следующее:

Атрибут src установлен в ссылку на тему, а не на URL изображения.
Все URL в атрибуте srcset исчезли (вероятно, возвращаются пустые значения).
Ни одно из оптимизированных изображений фактически не было сгенерировано (вероятно, из-за пустых URL).

Вот пример HTML, который работает корректно:

<img
    src="https://xxx.blob.core.windows.net/support/optimized/2X/2/2332e15b99ee86620e2f890d38aaceb16954cc8c_2_375x500.jpeg" 
    alt="20200113_163717" data-base62-sha1="51nILqcSQo5kGA4iz67LGoQmSpm"
    class="d-lazyload"
    srcset="//xxx.blob.core.windows.net/support/optimized/2X/2/2332e15b99ee86620e2f890d38aaceb16954cc8c_2_375x500.jpeg, //xxx.blob.core.windows.net/support/optimized/2X/2/2332e15b99ee86620e2f890d38aaceb16954cc8c_2_562x750.jpeg 1.5x, //xxx.blob.core.windows.net/support/optimized/2X/2/2332e15b99ee86620e2f890d38aaceb16954cc8c_2_750x1000.jpeg 2x" 
    width="375" height="500"
>

Для этого изображения всё работало корректно.

Код, отвечающий за генерацию оптимизированных изображений, находится в UrlHelper.cook_url, который претерпел серьёзные изменения для поддержки защищённых загрузок в S3.

NotAnonymous · 31.Январь.2020 14:40:43

Спасибо за информацию, я в итоге исправил это, основываясь на ваших комментариях.

Что касается недавних изменений, я тоже это обнаружил. Это было легко исправить: нужно просто добавить параметр secure (четвёртый) в функцию optimize в плагине. Не уверен, нужны ли другие изменения, но это устранило непосредственную проблему, которую вы указали.

schungx · 31.Январь.2020 14:43:26

С момента обновления примерно четыре дня назад у всех optimized_images поле url установлено в null вместо корректного URL изображения.

Из-за этого Discourse работает с серьёзными сбоями: upload.thumbnail возвращает корректную запись, но поле url равно null. Не должно ли где-то быть защитное условие для проверки этого?

Это приводит к тому, что следующая строка:

github.com/discourse/discourse

lib/cooked_post_processor.rb

d07f03946


      
              .sort
              .each { |r| yield r if r > 1 }
          end
          
          def optimize_image!(img, upload, cropped: false)
            w, h = img["width"].to_i, img["height"].to_i
          
            # note: optimize_urls cooks the src and data-small-upload further after this
            thumbnail = upload.thumbnail(w, h)
            if thumbnail && thumbnail.filesize.to_i < upload.filesize
              img["src"] = thumbnail.url
          
              srcset = +""
          
              each_responsive_ratio do |ratio|
                resized_w = (w * ratio).to_i
                resized_h = (h * ratio).to_i
          
                if !cropped && upload.width && resized_w > upload.width
                  cooked_url = UrlHelper.cook_url(upload.url, secure: @post.with_secure_media?)
                  srcset << ", #{cooked_url} #{ratio.to_s.sub(/\.0$/, "")}x"

устанавливает img["src"] в null. Уверен, что это ни к чему хорошему не приведёт.

Все изображения в srcset оказываются пустыми, потому что:

github.com/discourse/discourse

lib/cooked_post_processor.rb

d07f03946


      
          
            srcset = +""
          
            each_responsive_ratio do |ratio|
              resized_w = (w * ratio).to_i
              resized_h = (h * ratio).to_i
          
              if !cropped && upload.width && resized_w > upload.width
                cooked_url = UrlHelper.cook_url(upload.url, secure: @post.with_secure_media?)
                srcset << ", #{cooked_url} #{ratio.to_s.sub(/\.0$/, "")}x"
              elsif t = upload.thumbnail(resized_w, resized_h)
                cooked_url = UrlHelper.cook_url(t.url, secure: @post.with_secure_media?)
                srcset << ", #{cooked_url} #{ratio.to_s.sub(/\.0$/, "")}x"
              end
          
              img["srcset"] = "#{UrlHelper.cook_url(img["src"], secure: @post.with_secure_media?)}#{srcset}" if srcset.present?
            end
          else
            img["src"] = upload.url
          end

t.url оказывается равным null.

В итоге всё работает именно так, как есть сейчас. Уверен, что где-то по цепочке, если img["src"] равен null, он заменяется на URL самой темы.

Таким образом, корень проблемы в том, что оптимизированные изображения не создаются (хотя записи создаются, но с url равным null). Ошибок при этом не возникает.

Любые посты с изображениями, у которых есть корректные оптимизированные версии (с валидным url), работают без проблем, даже при перестроении HTML.

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

schungx · 31.Январь.2020 14:45:14

ИЗМ: ОК, я добавил secure: false в store_optimized_image, и теперь всё, кажется, работает нормально.

schungx · 31.Январь.2020 16:34:14

Я по-прежнему считаю, что необходимо что-то сделать для обработки ситуаций, когда у OptimizedImage поле url равно "" (а не null).

Это всё ещё будет происходить, но, похоже, только для некоторых изображений, а не для всех. Для определённых изображений такого рода миниатюры, создаваемые при использовании плагина Topics List Preview, не будут генерироваться, что приведёт к появлению записей с url="".

Обычно они не отображаются, так как адаптивные изображения начинаются с минимального размера 1x и увеличиваются (по умолчанию) до 2x. Поэтому к малым оптимизированным изображениям обращение никогда не происходит. Тем не менее, это вызывает беспокойство, что однажды это может привести к сбою…

Это не означает, что превью-изображение не будет отображаться в списке тем при установленном плагине Topics List Preview.

schungx · 22.Май.2020 05:50:32

При обновлении Discourse возникла ошибка из-за конфликта с faraday:


I, [2020-05-22T05:46:54.081847 #1]  INFO -- : > cd /var/www/discourse && su discourse -c 'bundle exec rake db:migrate'
rake aborted!
Gem::ConflictError: Unable to activate faraday_middleware-0.11.0, because faraday-1.0.1 conflicts with faraday (>= 0.7.4, < 1.0)
/var/www/discourse/lib/plugin_gem.rb:20:in `load'
/var/www/discourse/lib/plugin/instance.rb:635:in `gem'
/var/www/discourse/plugins/discourse-azure-blob-storage/plugin.rb:10:in `activate!'
/var/www/discourse/lib/plugin/instance.rb:541:in `instance_eval'
/var/www/discourse/lib/plugin/instance.rb:541:in `activate!'
lib/discourse.rb:224:in `block in activate_plugins!'
lib/discourse.rb:221:in `each'
lib/discourse.rb:221:in `activate_plugins!'
/var/www/discourse/config/application.rb:297:in `block in <class:Application>'
/var/www/discourse/lib/plugin_initialization_guard.rb:5:in `plugin_initialization_guard'
/var/www/discourse/config/application.rb:296:in `<class:Application>'
/var/www/discourse/config/application.rb:65:in `<module:Discourse>'
/var/www/discourse/config/application.rb:64:in `<top (required)>'
/var/www/discourse/Rakefile:7:in `require'
/var/www/discourse/Rakefile:7:in `<top (required)>'
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/rake-13.0.1/exe/rake:27:in `<top (required)>'
/usr/local/bin/bundle:23:in `load'
/usr/local/bin/bundle:23:in `<main>'

Использует ли discourse-azure-blob-storage faraday?

itsbhanusharma · 22.Май.2020 05:57:32

Да, действительно

github.com/discourse/discourse-azure-blob-storage

plugin.rb

67d13a285


      
          # name: discourse-azure-blob-storage
          # about: Azure Blob storage
          # version: 0.0.2
          # authors: Maja Komel
          # url: https://github.com/discourse/discourse-azure-blob-storage
          
          require "file_store/base_store"
          
          # GEMS
          gem 'faraday_middleware', '0.11.0', {require: false}
          gem 'azure-core', '0.1.13', {require: false}
          gem 'azure-storage-common', '1.0.1', {require: false}
          gem 'azure-storage-blob', '1.0.1', {require: false}
          
          require 'azure/storage/blob'
          
          enabled_site_setting :azure_blob_storage_enabled
          
          after_initialize do

schungx · 22.Май.2020 06:38:15

Отлично.

Это исправляет проблему.

meglio · 08.Август.2020 14:08:44

Этот плагин всё ещё не работает?

Кто-нибудь сейчас использует этот плагин для загрузки в Azure Blob Storage? Если да, то он стабилен? Поделитесь, пожалуйста, своим опытом.

Тема		Ответов	Просм.
Plugin to store uploads into Azure blob storage Feature	14	2036	23.02.2018
Wrong images link after cooking - since recent upgrade Support	5	1066	02.02.2020
[Not implemented] Error after upload any new image with azure blob storage Bug	5	1020	04.08.2021
All image uploads to Azure return Internal Server Error Support	6	1016	24.08.2023
Need Azure blob storage Feature	2	1249	31.08.2021

Azure Blob Storage Plugin

Связанные темы