Einbettung ist in 3.0.4 (stabil) DEFEKT

[fabsh (Fabian A. Scherschel)]

Ich habe den Verdacht, dass diese Änderung

Canonical url not being used for topic embeddings

die Art und Weise, wie ich Einbettungen auf meiner Website verwende, beeinträchtigt hat. Kann mir jemand sagen, was genau geändert wurde?

Ich verwende Discourse, um Themen für Beiträge auf meinem Blog zu erstellen (als Kommentarsystem). Meine Website ist für zwei Sprachen in zwei Teile unterteilt, daher verwende ich zwei Hosts, um die englischen (/blog/somepost) und deutschen (/de/blog/somepost) Beiträge in verschiedene Forenkategorien zu filtern.

Beiträge von der deutschen Seite der Website werden seit dem Update meines Discourse nicht mehr eingebettet. Ich vermute, es hat etwas mit dem obigen Patch zu tun.

Jede Hilfe bei der Lösung dieses Problems wäre sehr willkommen.

[fabsh (Fabian A. Scherschel)]

Andererseits könnte es sich um dieses Problem handeln:

Discourse Comments stuck on "Loading..."

Hello, Over the past few days, in our Discourse, we have noticed that the “View Full Post” button remains at “Loading…” when it’s clicked: [image] Additionally, new Discourse comments have stopped showing from our embed from within our Drupal website. We have been successfully using the JavaScript embed instructions from below for years: However, for some reason this seems to have stopped working lately. I believe the last working post was around the 1st of this month. Older posts do displ…

Ich habe sicherlich diese DOMExceptions, die angezeigt werden. Nochmals, jede Hilfe wäre sehr willkommen!

[Lilly (Lillian Louis)]

Ja, ich würde mir die Sicherheitsrichtlinien und -einstellungen für Ihr Forum und die eingebettete Website ansehen. Der Screenshot sieht in Ordnung aus. Welches Skript verwenden Sie und was ist mit Ihren anderen Einbettungseinstellungen? Haben Sie dort kürzlich Änderungen vorgenommen?

Mein Forum nutzt diese Einbettungs-Posting-Funktion mehrmals täglich und sie funktioniert im Moment großartig, daher weiß ich, dass sie nicht kaputt ist. Wir verwenden die neueste Betaversion und sind gehostet, daher bin ich zuversichtlich in die Konfiguration.

[fabsh (Fabian A. Scherschel)]

@Lilly

Ja, ich würde mir die Sicherheitspolicys und Einstellungen für Ihr Forum und die eingebettete Website ansehen. Der Screenshot sieht in Ordnung aus. Welches Skript verwenden Sie und was ist mit Ihren anderen Einbettungseinstellungen? Haben Sie dort kürzlich Änderungen vorgenommen?

Hallo, danke, dass du versuchst zu helfen!

Ich habe keine Kontrolle über Sicherheitspolicys, da beide Seiten von Hosting-Anbietern gehostet werden. Ich bezweifle irgendwie, dass sie das kaputt gemacht haben. Aber ich werde das mit meinem Discourse-Hoster untersuchen.

Ich verwende dieses Skript im Blog (es ist eine statische Website-Engine namens Hugo):

<script type="text/javascript">
  DiscourseEmbed = { discourseUrl: 'https://forum.fab.industries/',
                     discourseEmbedUrl: '{{ .Permalink }}' };

  (function() {
    var d = document.createElement('script'); d.type = 'text/javascript'; d.async = true;
    d.src = DiscourseEmbed.discourseUrl + 'javascripts/embed.js';
    (document.getElementsByTagName('head')[0] || document.getElementsByTagName('body')[0]).appendChild(d);
  })();
</script>

Ich habe nichts davon geändert, seit ich es vor ein paar Monaten zum ersten Mal integriert habe.

Alles, was ich geändert habe, war etwas kleines benutzerdefiniertes CSS, das ich im Forum-Theme hinzugefügt habe. Ich werde das heute rückgängig machen und sehen, ob das Problem dadurch behoben wird.

[fabsh (Fabian A. Scherschel)]

Okay. Das Zurückrollen des CSS hat nichts gebracht. Es war sowieso ein Schuss ins Blaue.

Ich konnte jetzt auch bestätigen, dass das Einbetten generell kaputt ist. Die englische Seite des Blogs funktioniert auch nicht mehr. Neue Seiten bleiben bei “Diskussion wird geladen…” hängen.

Ich bin mir jetzt ziemlich sicher, dass dies kaputt ging, als mein Anbieter mein Forum auf 3.0.4 aktualisiert hat, aber ich weiß nicht, von welcher Version sie mich aktualisiert haben. Daher vermute ich immer noch diesen Patch:

FEATURE: Update topic/comment embedding parameters (#20181)

Ich denke, die Tatsache, dass der Browser diese DOM-bezogenen Fehler ausgibt, ist kein Zufall:

Das Deaktivieren von CSP in den Admin-Einstellungen scheint es nicht zu beheben.

[RGJ (Richard - Communiteq)]

So main has this

TopicEmbed.import_remote(@embed_url, user: User.find_by(username_lower: username.downcase))

and stable has this

TopicEmbed.import_remote(user, @embed_url)

Note the order of the parameters.

Now the backport of the security patch changed the function signature on stable to the new parameter order,so

def self.import_remote(import_user, url, opts = nil)

became

def self.import_remote(url, opts = nil)

and now the url parameter receives a User object.

Changing the function call resolves the issue

diff --git a/lib/topic_retriever.rb b/lib/topic_retriever.rb
index b798df6cd7..6186ce5868 100644
--- a/lib/topic_retriever.rb
+++ b/lib/topic_retriever.rb
@@ -50,6 +50,6 @@ class TopicRetriever
     user = User.where(username_lower: username.downcase).first
     return if user.blank?

-    TopicEmbed.import_remote(user, @embed_url)
+    TopicEmbed.import_remote(@embed_url, user: user)
   end
 end

@blake

[RGJ (Richard - Communiteq)]

Ich habe einen PR eingereicht, der dieses Problem behebt FIX broken topic embedding because of incomplete security patch (#22088) by communiteq · Pull Request #22184 · discourse/discourse · GitHub

[pmusaraj (Penar Musaraj)]

Danke @RGJ für die Korrektur, dieser PR ist jetzt gemerged.