Изображения в Onebox с неверным соотношением сторон, когда oEmbed указывает размеры для встраивания в богатый HTML

Размеры берутся из oembed:

https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F

В ответе содержится:

Я полагаю, что ошибка заключается в следующем: если тип — “rich”, и мы не извлекаем всю презентацию из полезной нагрузки html oembed, то мы должны пропустить добавление размеров, так как эти данные нам не интересны.

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

diff --git a/lib/onebox/engine/standard_embed.rb b/lib/onebox/engine/standard_embed.rb
index e3175d6247..fc8c300d81 100644
--- a/lib/onebox/engine/standard_embed.rb
+++ b/lib/onebox/engine/standard_embed.rb
@@ -159,8 +159,9 @@ module Onebox
         @json_ld ||= Onebox::JsonLd.new(html_doc)
       end
 
-      def set_from_normalizer_data(normalizer)
+      def set_from_normalizer_data(normalizer, skip_dimensions: false)
         normalizer.data.each do |k, _|
+          next if skip_dimensions && k.in?(%i[width height])
           v = normalizer.public_send(k)
           @raw[k] ||= v unless v.nil?
         end
@@ -179,7 +180,8 @@ module Onebox
 
       def set_oembed_data_on_raw
         oembed = get_oembed
-        set_from_normalizer_data(oembed)
+        skip_dimensions = oembed.data[:type] == "rich"
+        set_from_normalizer_data(oembed, skip_dimensions:)
       end
 
       def set_json_ld_data_on_raw

Однако я не уверен, какие ещё побочные эффекты это может вызвать, поэтому отмечаю команду member-experience, которая рассмотрит это в течение следующего месяца.

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


Приоритет — средний, так как влияние этой ошибки довольно широкое, учитывая, что wp-json предоставляет эти данные.

1 лайк