Joomla 4.2.6. Email-Verschleierung funktioniert teils nicht mehr.

Re:Later · 15. Dezember 2022

Ist schon eingereicht bei Joomla- Nur als Hinweis, weil's bei "meinen" Seiten (von Kunden) z.B. teils gravierend ist, dass die Emails jetzt offen liegen:

Das Verschleiern (Cloaking) von Email-Adressen, also, dass sie für Spammer-Bots nicht so leicht auslesbar sind, funktioniert in einigen Szenarien nicht mehr. Die Email-Adressen sind auch ohne JavaScript auslesbar, wo ja im Normalfall dann stehen sollte "Diese Adresse ist vor Spam-Bots geschützt".

Und zusätzlich: Email-Adressen, die zuvor von Joomla automatisch klickbar verlinkt wurden, wenn noch nicht zuvor, liegen als unklickbarer, blanker Text vor.

Mein Fix (und weiteres im Thread): RE: Automatische E-Mail Generierung funktioniert nicht (tp_snowflake)

LukasHH · 16. Dezember 2022

Einen erste Lösungsansatz habe ich im Issiu ergänzt. Wenn Du magst, dann kannst Du das vorab schon einmal testen.

christine2 · 16. Dezember 2022

Hallo Peer & Re:Later

hab's versucht - wird wohl beim (erfolglosen) Versuch bleiben. Sicher ev. auch was falsch gemacht

Vorheriger Code, Zeile 444:

Code

$pattern = '~<[^<]*(?<!\/)>(*SKIP)(*F)|(<\w.*\"' . $searchEmail . '\".*\/\>)~i';

geändert auf:

Code

$pattern='~<[^<]*(?<!\/)>(*SKIP)(*F)|(<\w.*\"' . $searchEmail . '\".*\/\>)|' . $searchEmail . '~i

wobei mir da auffällt, dass beim geänderten kein ; am Ende ist. Beim ursprünglichen aber schon.
Hab ich aber dazugegeben. Ev. nicht machen sollen.

Erfolg ist ein 404er (Startseite):

0 syntax error, unexpected identifier "email"

Liebe Grüße

Christine

JoomlaWunder · 16. Dezember 2022

christine2: Muss da nicht '; ans Ende?

christine2 · 16. Dezember 2022

Zitat von JoomlaWunder

christine2: Muss da nicht '; ans Ende?

Habe ich ja geschrieben, dass ich ein ; dazugegeben habe.

Zitat von christine2
geändert auf:
Code
$pattern='~<[^<]*(?<!\/)>(*SKIP)(*F)|(<\w.*\"' . $searchEmail . '\".*\/\>)|' . $searchEmail . '~i
wobei mir da auffällt, dass beim geänderten kein ; am Ende ist. Beim ursprünglichen aber schon.
Hab ich aber dazugegeben. Ev. nicht machen sollen.

LukasHH · 16. Dezember 2022

Die komplette Zeile müsste so aussehen:

PHP

$pattern = '~<[^<]*(?<!\/)>(*SKIP)(*F)|(<\w.*\"' . $searchEmail . '\".*\/\>)|' . $searchEmail .'~i';

christine2 · 16. Dezember 2022

Vorhin (vor Deinem Post) nochmals probiert, natürlich mit dem ; zum Schluss, bekam aber nun andere Meldung:

syntax error, unexpected token "<"

Vergleiche es mal mit einem Bildchen:

Da dürfte ein ' fehlen. Das von Issue genommen. OK - bis gleich. ......

LukasHH · 16. Dezember 2022

Ja - die Text-Anfang/Ende fehlen. Daher hatte ich die komplette Zeile nochmal geschrieben. Aber das läuft bei mir auf TimeOut.
Ich versuche gerade noch was anderes.

christine2 · 16. Dezember 2022

Die beiden ' nun ergänzt. Unsere Mails haben sich überschnitten.

Also wieder: 0 syntax error, unexpected identifier "email" und Ja, auch ein Time Out, wie ich gerade sehe.

Re:Later · 16. Dezember 2022

Nur nebenbei: Auch meine Codes auf Github sind abgeschnitten bzgl. der finalen Quotes ('). Keine Ahnung, ob's an mir lag/liegt.

Re:Later · 16. Dezember 2022

Zitat von LukasHH

$pattern = '~<[^<]*(?<!\/)>(*SKIP)(*F)|(<\w.*\"' . $searchEmail . '\".*\/\>)|' . $searchEmail .'~i';

Ein Timeout bekomme ich mit

Code

<a href="#">info@example.com</a>

aber nicht mit

Code

<p>info@example.com</p>

Aber bei zweiterem "matcht" das Pattern auch gar nicht

LukasHH · 16. Dezember 2022

Das Pattern in der Zeile 444 ist ja für Einträge wie diese hier gedacht gewesen:

HTML

<img src="..." title="email@example.org">
<input type="text" placeholder="email@example.org">

Da dieses nicht funktionierte, hatte ich das Pattern angepasst.

Man müsste ein zusätzliches Pattern wie vorher in 4.2.5 erstellen, damit diese Form funktioniert

HTML

<p>email@example.org</p>

Was aber dabei noch nicht richtig funktioniert ist das folgende Beispiel:

HTML

<a title="email@example.org" href="#">email@example.org</a>

Beim letzten wird die zweite Mail-Adresse erkannt, jedoch aber nicht die im title-Attribut. Die bleibt weiterhin im Klartext stehen. Ich werde da noch was anderes probieren.

LukasHH · 27. Dezember 2022

Ich habe ein PR erstellt. Wenn Ihr möchtet, dann gerne einmal testen:

[4.2.6] Email cloaking fails. by LukasHH · Pull Request #39508 · joomla/joomla-cms

Pull Request for Issue #39422 . Summary of Changes a) modification of the last pattern b) added more patterns Testing Instructions create article with for the…

github.com