Broken HTML when embedding tweet

[abid76]

First, thanks for this great plugin 🙂

We noticed that sometimes Embed Privacy breaks HTML structure when embedding tweets. The HTML is all right when disabling Embed Privacy.

Unfortunately, I couldn't reproduce the issue in my staging environment, even with exactly the same post content and the same Embed Privacy version installed.

You can see the issue here. The issue shows up near the words lange nachdem Maccabi schon wieder abgereist ist. Please note there's an a element preceding the embed. This may be causing the broken HTML.

We're using WP version 6.6.2. Updating the plugin to current version 1.10.6 didn't help. Maybe this is related to the actual libxml version used, but that's just a guess.

Is it a bug in Embed Privacy? Is there any additional information we shall provide to investigate this further?

[MatzeKitt]

Can you share the code of this post in backticks here, like this:

Code

[MatzeKitt]

Hm, now the link is identical for me.

Currently, I simply don’t know where to even start debugging, since I cannot reproduce the issue. However, when don't allow anything in the CMT, only plain Twitter links are displayed. Maybe your CMT is manipulating something, which then produces code Embed Privacy cannot process.

Can you share the source code of the page with the following variants (in the frontend):

1. Embed Privacy enabled + CMT enabled
2. Embed Privacy enabled + CMT disabled (completely on server-side)
3. Embed Privacy disabled + CMT enabled
4. Embed Privacy disabled + CMT disabled (completely on server-side)

[abid76]

Here you are (I suppose you mean cookie banner by "CMT"). It's really strange. Now everything is displayed correctly when EP is enabled, but not when EP is disabled.

There has been a WP upgrade from 6.6 to 6.7 since the start of this thread.

Twitter Test.zip

[MatzeKitt]

Yes, by CMT I meant your cookie banner aka consent management tool.

If it’s now working fine with Embed Privacy and not properly without, Embed Privacy cannot be the issue. 😬

[abid76]

Sorry, but the issue now shows up on a different post. Now it does seem to be caused by EP. The layout is only broken when EP is enabled. You can see it here (layout is broken after "weil sie eine Frau ist").

I updated the uploaded source codes with the HTML sources.

Here's the source code of the post:

<!-- wp:paragraph -->
<p>„Weggucken ist verlächerlichen, verniedlichen, gehört sich nicht, geht nicht“, sagt Robert Habeck am Montag mit in Falten gelegter Stirn in die Kamera. Der Minister nutzt den „Orange Day“, den Tag gegen Gewalt an Frauen, um mit hochgekrempelten Ärmeln und Hundeblick sein Image aufzupolieren. Endlich kann er sich nicht mehr nur zum Frauenschwarm, sondern auch zu unserem Retter aufstilisieren. Frei nach dem Motto: Frauen dieser Welt schaut auf diesen Mann! Robert lässt euch nicht im Stich, Robert kämpft für euch – Robert wird euch alle retten! Oder anders gesagt: Robert heuchelt euch die Hucke voll.&nbsp;</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>Denn was Habeck hier macht ist nicht weniger, als genau das, was er anprangert: Er guckt weg, er verlächerlicht. Wie der Großteil seiner Kollegen aus dem Bundestag – von Annalena Baerbock über Franziska Brandner bis hin zu Katharina Dröge – stimmt Habeck nämlich nur in den immer gleichen Sprechchor ein: Das eigene Zuhause ist für Frauen der gefährlichste Ort. Wir brauchen mehr Frauenhäuser, Beratungs- und Nothilfeprogramme. Nur so können wir dieses Problem „in unserer Gesellschaft“ lösen. Nur so können wir verhindern, dass „fast jeden Tag“ eine Frau getötet wird, „weil sie eine Frau ist“. </p>
<!-- /wp:paragraph -->

<!-- wp:embed {"url":"https://twitter.com/roberthabeck/status/1860970223701999841","type":"rich","providerNameSlug":"twitter","responsive":true} -->
<figure class="wp-block-embed is-type-rich is-provider-twitter wp-block-embed-twitter"><div class="wp-block-embed__wrapper">
https://twitter.com/roberthabeck/status/1860970223701999841
</div></figure>
<!-- /wp:embed -->

<!-- wp:paragraph -->
<p>Da ist sich Habeck sogar mit Friedrich Merz einig – die beiden scheinen zu üben, wie eine schwarz-grüne Koalition in Bund künftig aussehen könnte. Den gleichen Social Media-Stil haben sie jedenfalls schonmal: Auch Friedrich Merz blickt betroffen mit tiefen Dackelfalten in die Kamera, während er einen „nationalen Aktionsplan“ zum besseren Schutz von Frauen fordert. Er will elektronische Fußfesseln, verpflichtende Anti-Aggressionstrainings und Prävention. Das Wörtchen Migration nimmt er nicht in den Mund – Mutti Merkel würde das nicht gefallen.</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>Es ist jedes Jahr die gleiche Leier: Wir reden über „Femizide“ ohne auch nur einmal zu thematisieren, in welchen Kulturkreisen sie am häufigsten vorkommen – warum es früher „Ehrenmord“ hieß. Wir sprechen nicht darüber, warum in Wien ein zehn Tage alter türkischer Säugling von seiner Mutter erschlagen wurde oder warum ein libanesischer Mann im August in Berlin seine Ex-Frau auf offener Straße mit einem Messer hinrichtete (<em>Apollo News</em> <a href="https://apollo-news.net/ehrenmord-frau-in-berlin-zehlendorf-von-ex-mann-mit-messer-ermordet/">berichtete</a>). </p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>Wir nehmen die Namen von Frauen wie Hatun Sürücü, die von ihren Brüdern getötet wurde, weil sie sich ein modernes Leben wünschte, nicht mehr in den Mund. Und lassen bei der viel zitierten Statistik der Vereinten Nationen einfach den Fakt weg, dass die mit Abstand meisten „Femizide“ nicht in Europa, sondern in Afrika stattfinden. </p>
<!-- /wp:paragraph -->

<!-- wp:embed {"url":"https://twitter.com/_FriedrichMerz/status/1860741943619101089","type":"rich","providerNameSlug":"twitter","responsive":true} -->
<figure class="wp-block-embed is-type-rich is-provider-twitter wp-block-embed-twitter"><div class="wp-block-embed__wrapper">
https://twitter.com/_FriedrichMerz/status/1860741943619101089
</div></figure>
<!-- /wp:embed -->

<!-- wp:paragraph -->
<p>Als wäre das noch nicht genug, schmeißen wir dann auch noch die sexuelle Gewalt gegen Frauen, die sich zumeist in der Öffentlichkeit abspielt, mit „Femiziden“ und Partnerschaftsgewalt in einen Topf. Auch Habeck nennt die Begriffe in einem Atemzug. Er beklagt: „Täglich werden 140 Frauen Opfer einer Sexualstraftat“ und schämt sich nicht, wenig später die Worte folgen zu lassen, dass „über solche Taten, über solche Übergriffe (…) ja ganz selten offen geredet“ wird. </p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>Dann ändert sich der Ton ins Vorwurfsvolle. Habeck ist sauer, dass das Ampel-Aus und die Neuwahlen seinen schönen Plan durchkreuzt haben, das „Gewalthilfegesetz“ im Bundestag durchzudrücken. Dabei wollte er doch so gerne mit dem neuen Paragraphen am Revers für ein „Frauen-schützendes“ Land eintreten.</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>Stattdessen konnte Habeck jetzt nur mit seinem traurigen Gesicht ein Zeichen setzen – aber das ist bei unserer Bundesregierung ja sowieso viel wichtiger, als Tatsachen zu schaffen. Immerhin, da ist man ehrlich. Warum sollte man sich auch mit so unangenehmen Fakten auseinandersetzen, wie dem, dass Täter aus arabischen Ländern wie Syrien, Afghanistan und dem Irak bei Sexualstraftaten gemessen an ihrem Bevölkerungsanteil überrepräsentiert sind (Drucksache 20/11412, Stand 2023)?</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph -->
<p>Man kann schließlich auch einfach Statuen mit orangenen Mützen ausstatten, Flaggen hissen, Bröttchentüten gegen Gewalt verteilen oder Bänke orange anmalen – <a href="https://www.ndr.de/nachrichten/niedersachsen/Orange-Day-2024-Aktionen-in-Niedersachsen-gegen-Gewalt-an-Frauen,orangeday136.html">Niedersachsen</a> macht vor. Dann haben alle ein gutes Gefühl – alle, außer die Frauen, die nachts allein um ihr Leben und ihre körperliche Unversehrtheit fürchten. Aber um die geht es Habeck und seinen Kollegen ja sowieso nicht.</p>
<!-- /wp:paragraph -->

[MatzeKitt]

What about the status of the cookie banner? To me, it still looks like a problem related to it.