[#36981] - [4.0] [3.10] E-Mail Cloak: part of the TLD will be truncated
- Closed
- 9 Feb 2022
- Medium
- Build: 4.0
- # 36981
Steps to reproduce the issue
Reported by Joomla Forum: https://forum.joomla.de/thread/15298-email-adresse-mit-doppeltem-link-dargestellt/
Expected result
Actual result
Once the plugin is actived, part of the TLD will be truncated. Example:
office@some.international
becomes:
office@some.internatio
System information (as much as possible)
Joomla 4.0.6
Additional comments
could be relevant:
// Example: any@example.org
$searchEmail = '([\w.'-+]+@(?:[a-z0-9.-]+.)+(?:[a-zA-Z0-9-]{2,10}))';
| Labels |
Removed:
?
|
||
joomla-cms-bot
- | Labels |
Added:
No Code Attached Yet
|
||
I can confirm this issue, TLD 'international' seems to be a valid TLD. Increasing 2,10 to 2,13 does no longer truncate this TLD.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36981.
I can confirm this issue, TLD 'international' seems to be a valid TLD. Increasing 2,10 to 2,13 does no longer truncate this TLD.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36981.
@ChrisHoefliger Thanks for your investigation!
@brianteeman any PR possible? Thanks
The bug fix should then also be in Joomla 3.10. This completely outdated RegEx is also used there.
There hasn't been a length limitation for a long time now except >= 1.
| Title |
|
||||||
Anyone can submit a pull request
chmst
- | Labels |
Added:
bug
|
||
richard67
- | Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-02-09 21:29:14 |
| Closed_By | ⇒ | richard67 |
its the 2,10 part of the regex causing the truncation so it is simple to fix by increasing the number