Les dimensions proviennent de oembed :
https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F
Ce qui a :
Je pense que le bug ici est que si le type est « rich », et que nous ne récupérons pas la présentation entière du payload html de oembed, nous devrions ignorer l’ajout des dimensions, car nous ne nous intéressons pas à ces données.
Ceci corrige le problème :
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
Cependant, je ne suis pas sûr des autres effets secondaires que cela pourrait avoir, j’alerte l’équipe chargée de l’expérience membre qui examinera cela au cours du mois prochain.
Je suis réticent à simplement ajouter mon correctif car il y a beaucoup de couches et de complexité ici, quelqu’un doit s’assurer que nous pouvons ajouter le correctif de manière très sûre et testée.
Ceci est pri-moyen car l’impact de ce bug est assez large étant donné que wp-json le met en évidence.