Problème avec Onebox et YouTube

Contribuer Bug
1

par exemple

La miniature YouTube n’apparaît pas dans la liste des sujets, par exemple avec le thème Fakebook ou le plugin TLP.

Scénario de reproduction (STR) :

Publier une vidéo YouTube, passer au thème Fakebook : aucune miniature n’apparaît.

Solution de contournement :

Reconstruire le sujet fait réapparaître la miniature. Cependant, étant donné que les utilisateurs s’attendent à ce que la miniature s’affiche immédiatement sans intervention du personnel, ce n’est pas une solution idéale.

Plus d’informations :

Topic.image_url semble être nil.

Cela semble s’être cassé récemment, début août… Je pense que cela fonctionnait avant le 5 août (mais je ne suis pas sûr de la date exacte de la rupture). J’ai remarqué que cela correspond à peu près à une nouvelle version de Onebox…

J’ai testé cela sur une version précédente et le problème n’était pas présent, ce qui élimine l’hypothèse d’une réponse og de YouTube. De plus, le thème Fakebook présente le même comportement que le plugin TLP, ce qui élimine également ce dernier. Je soupçonne donc une régression récente dans Discourse ou Onebox.

C’est un vrai problème pour les utilisateurs des aperçus de la liste des sujets.

4 « J'aime »
2

Mise à jour : j’ai effectué un Git Bisect et il semble que ce soit le commit problématique :

3 « J'aime »
3 
diff --git a/lib/cooked_post_processor.rb b/lib/cooked_post_processor.rb
index ac16eb82c16..46050a69e40 100644
--- a/lib/cooked_post_processor.rb
+++ b/lib/cooked_post_processor.rb
@@ -28,7 +28,7 @@ def initialize(post, opts = {})
     @cooking_options = @cooking_options.symbolize_keys
 
     @doc = Nokogiri::HTML::fragment(post.cook(post.raw, @cooking_options))
-    @has_oneboxes = post.post_analyzer.found_oneboxes?
+    @has_oneboxes = @doc.css("aside.onebox").count > 0
     @size_cache = {}

@nbianca pourquoi n’utilisons-nous plus la méthode post.post_analyzer.found_oneboxes? ici ?

Tous les oneboxes utilisant soit oembed soit iframe échoueront avec le nouveau mode de sélecteur CSS, ce qui rend également notre détecteur de Onebox différent dans l’ensemble du code.

4 « J'aime »
4

Je suis d’accord avec @nbianca, cela ressemble à une modification extrêmement fragile. Pourquoi cette méthode a-t-elle été choisie ?

2 « J'aime »
5

Le problème était que le First Onebox était attribué pour tout lien publié, qu’il s’agisse ou non d’un Onebox. Cela se produit parce que PrettyText ajoute class="onebox" à tous les liens, puis Oneboxer tente d’analyser tous les liens ayant la classe onebox.

La modification que j’ai apportée vérifie si le document final contient des Oneboxes (les Oneboxes sont des aside avec la classe onebox). Je vais examiner de plus près la situation pour comprendre ce qui est cassé et pourquoi, et je reviendrai avec une correction qui résoudra les deux problèmes.

7 « J'aime »
6

J’ai apporté quelques améliorations ici et créé cette PR.

Je viens de tester la version 2.3.2 et il ne semble pas que les liens YouTube définissent l’attribut image_url du sujet. La raison en est que l’image est extraite de l’attribut src d’une balise img, mais lorsqu’un lien vidéo YouTube est ajouté, il y a un div avec un attribut CSS background-image.

Êtes-vous certain que image_url était défini auparavant ?

5 « J'aime »
7

Merci d’avoir regardé cela. Laissez-moi vérifier de mon côté et je reviens vers vous.

8

@nbianca désolé, il s’agit peut-être d’une combinaison de deux problèmes.

Fonctionne avant et jusqu’à cette version avec le plugin TLP :

Après ce commit, cela échoue avec TLP.

Je suis d’accord avec vous : pour YouTube, cette propriété n’est pas remplie par défaut.

Il semble donc que TLP gérait mieux ses surcharges, mais en s’appuyant sur quelque chose qui a maintenant changé.

Pour information, les surcharges concernées se trouvent ici : https://github.com/angusmcleod/discourse-topic-previews/blob/master/lib/cooked_post_processor_edits.rb

Je pense que je devrai effectuer une analyse plus approfondie, mais il semble que des changements suffisants aient eu lieu pour rompre la compatibilité avec TLP.

Je vais creuser un peu plus et revenir en arrière. Malheureusement, je ne serai peut-être pas disponible au cours des prochains jours.

1 « J'aime »
9

Je ne vois pas comment ce commit peut casser le plugin TLP :thinking: Êtes-vous sûr à 100 % de votre bisect ?

1 « J'aime »
10

C’est très étrange que le bisect ait trouvé un changement dans cooked_post_processor… Je vais vérifier à nouveau et annuler.

Oui, confirmé :

Avec ce commit, cela échoue : 7c83d2eeb261ac676a8320e6a704752c56fd242e
Avec ce commit précédent, tout va bien : 922c40f87cdceb8197dd61361334e0def23f13d5

1 « J'aime »
11

Le problème a été résolu ici :

7 « J'aime »