StringTool.isMailAddress

[rgwch]

Nachdem der Fehler in StringTool.Unique jetzt kaum 7 Jahre nach der Meldung in Redmine #3822 bereits behoben wurde ;-) wage ich nochmal einen ähnlich aufwendigen Verbesserungsvorschlag:

Der Matcher in isMailAdress() lautet bisher:

  "\\w[\\w|\\.\\-]+@\\w[\\w\\.\\-]+\\.[a-zA-Z]{2,7}"

müsste aber sein:

    "\\w[\\w|\\.\\-\\+]+@\\w[\\w\\.\\-]+\\.[a-zA-Z]{2,7}"

Denn das + Zeichen ist gültig in Mailnamen, z.B. gibt es ja in Google Mail die Strukturierungsoption:

[email protected]
[email protected]
usw.

die alle valide Adressen sind, die auf denselben Account geleitet werden.

[col-panic]

Es gibt auch in er OWASP Registry einen empfohlenen RegEx String https://owasp.org/www-community/OWASP_Validation_Regex_Repository für e-mail validation.

[rgwch]

Das wäre dann möglicherweise noch besser. Bin nur darüber gestolpert, weil beim Testen eines automatischen Mailversands meine eigene Mailadresse (mit +) nicht akzeptiert wurde ;-)