Las dimensiones provienen de oembed:
https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F
Lo cual tiene:
Creo que el error aquí es que si el tipo es “rich” y no estamos capturando toda la presentación del payload html de oembed, deberíamos omitir la adición de dimensiones, ya que no nos interesa esta información.
Esto lo soluciona:
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
Sin embargo, no estoy seguro de qué otros efectos secundarios tendría esto, informando al equipo de experiencia de miembro que lo revisará durante el próximo mes.
Soy reacio a simplemente agregar mi parche porque hay muchas capas aquí y complejidad, alguien necesita asegurarse de que podamos agregar el parche de una manera muy segura y probada.
Esto es pri-medium porque el impacto de este error es bastante amplio dado que wp-json lo expone.