Project

General

Profile

Feature #7860

mailto Link (Antispam) - JS bricht nicht um an UTF8 Character Grenzen

Added by Benjamin Baer over 1 year ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
12.02.2019
Due date:
16.08.2019
% Done:

0%

Estimated time:
Discuss:

Description

Bei Gelegenheit updaten, jetzige Implementation erzeugt falsche Darstellungen mit Sonderzeichen.

Stack zum Thema:
https://stackoverflow.com/questions/483212/effective-method-to-hide-email-from-spam-bots

Aktuellste Quelle laut Stack:
http://www.grall.name/posts/1/antiSpam-emailAddressObfuscation.html

stringSplitProblem.png (57.2 KB) stringSplitProblem.png Benjamin Baer, 13.02.2019 10:08
2245

Associated revisions

Revision 9ab4c52c (diff)
Added by Carsten Rose 10 months ago

Fixes #7860. Shown Text is only splitted if it is a email address.

Revision af7f38b5 (diff)
Added by Carsten Rose 10 months ago

Refs #7860. Add unit test

History

#1 Updated by Carsten Rose over 1 year ago

  • Status changed from New to Feedback
  • Was ist der Hintergrund das du das jetzt ansprichst? Gibt es aktuellen Handlungsbedarf?
  • T3 Variante habe ich nicht in production angeschaut - aber die Doku suggeriert mit das das nicht sonderlich gut ist: https://docs.typo3.org/typo3cms/TyposcriptReference/8.7/Setup/Config/#spamprotectemailaddresses
  • Die beiden Links sind alt (2008 ... 2015) - die Javascript Loesung wird bei Link 1 noch mit 100% bewertet ... die Zeiten duerften lange vorbei sein.
  • Die Variante mit der User Interaction fand ich noch gut
  • Die jetzige Loesung muessen wir verbessern - es wird der Text zerstueckelt, nicht aber der mailto link - ich glaube das wurde gemacht um die teilweise Unterstreichung zu fixen.

Requirements/Wishes von meiner Seite:

  • Bots sollten es schwer haben die Adresse zu lesen.
  • Klick mit der Maus soll das Mailprogramm oeffnen.
  • Ein Markieren und copy (to clipboard) sollte moeglich sein.

Ich wuerde die alte Loesung wieder nehmen, nur die Trennung nicht nach 2 chars machen sondern um das herum - das hover dann nur den Teil links/rechts vom highlighted ... egal.

#2 Updated by Benjamin Baer over 1 year ago

2245

Hier sieht man es schoen, das die Sonderzeichen probleme beim Split machen. Nehme an, das ist weil die Sonderzeichen in UTF8 aus 2 bytes bestehen und der string split wohl C basierend bytes splited. (Eventuell gibts dafuer auch optionen)

Ich denke nicht das sich sich viel bei Bots geaendert hat seit 2008, die Studie aus 2015 zietiert die ja noch und schliesslich ist email ja eh nur fuer internet ausdrucker. ;)

#3 Updated by Benjamin Baer over 1 year ago

  • Due date changed from 27.02.2019 to 26.07.2019

#4 Updated by Carsten Rose about 1 year ago

  • Tracker changed from Support to Feature
  • Target version set to 146

#5 Updated by Benjamin Baer 11 months ago

  • Subject changed from Updating Antispam to mailto Link (Antispam) - JS bricht nicht um an UTF8 Character Grenzen

#6 Updated by Benjamin Baer 11 months ago

  • Due date changed from 26.07.2019 to 16.08.2019
  • Assignee changed from Benjamin Baer to Carsten Rose

#7 Updated by Carsten Rose 11 months ago

  • Status changed from Feedback to Rejected

CR dachte es haette mit der 'm:|e' zu tun - wenn encryption aktiviert ist.
Das ist aber nicht der Fall - das ist gar nicht abschliessend implementiert!!

Das Beispiel im Screenshot ist nicht klar wie es erzeugt wurde - wo wurde der Screenshot gemacht?

#8 Updated by Carsten Rose 11 months ago

  • Target version changed from 146 to 19.8.0

#9 Updated by Carsten Rose 10 months ago

  • Status changed from Rejected to In Progress
  • Target version changed from 19.8.0 to 146
  • Das Problem war nicht bei der Email, sondern beim angezeigten Namen.
  • Enthielt der Name als zweiten Character einen Umlaut, wurde dieser dur PHP substr(..., 2) zerrisen.
  • Neue Implementierung trennt den Namen nur noch wenn eine Emailadresse angegen ist. Bei einem normalen Text wird nur ein JS Part ausgegeben.

#10 Updated by Carsten Rose 10 months ago

  • Status changed from In Progress to Closed

#11 Updated by Carsten Rose 10 months ago

  • Target version changed from 146 to 19.9.0

Also available in: Atom PDF