# Imagens do Onebox têm proporção incorreta

**URL:** https://meta.discourse.org/t/onebox-images-with-bad-aspect-when-oembed-specifies-dimentions-for-rich-html-embeds/277098
**Category:** Bug
**Tags:** onebox
**Created:** [29 Agosto , 2023 20:05 UTC](https://meta.discourse.org/t/onebox-images-with-bad-aspect-when-oembed-specifies-dimentions-for-rich-html-embeds/277098 "2023-08-29T20:05:00Z")
**Posts on this page:** 1
**Showing post:** 13

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [7 Julho , 2025 04:46 UTC](https://meta.discourse.org/t/onebox-images-with-bad-aspect-when-oembed-specifies-dimentions-for-rich-html-embeds/277098/13 "2025-07-07T04:46:09Z")

</div>

As dimensões vêm do oembed:

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

```

Que tem:

 ![Esta imagem exibe uma página da web com conteúdo incorporado relacionado ao contato de desenvolvedores de outros aplicativos e informações de integração, conforme indicado pelo código HTML e título. (Legenda por IA)](https://global.discourse-cdn.com/meta/original/4X/7/4/e/74e0cf27f5c30e65e5c1538df506d7a25566e685.png)

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
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.

---

_[View the full topic](https://meta.discourse.org/t/onebox-images-with-bad-aspect-when-oembed-specifies-dimentions-for-rich-html-embeds/277098)._