As dimensões vêm do oembed:
https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F
Que tem:
Acho que o bug aqui é que, se o tipo for “rich”, e não estivermos capturando toda a apresentação do payload html do oembed, devemos pular a adição de dimensões, pois não estamos interessados nesses dados.
Isso corrige:
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
No entanto, não tenho certeza de quais outros efeitos colaterais isso teria, sinalizando a equipe de experiência do membro, que dará uma olhada nisso no próximo mês.
Estou relutante em simplesmente adicionar meu patch, pois há muitas camadas e complexidade aqui, alguém precisa garantir que possamos adicionar o patch de uma maneira muito segura e testada.
Isso é pri-médio, pois o impacto desse bug é bastante amplo, dado que o wp-json o expõe.