-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#33948] fix email validation, allowing email with new domain name extension #4135
Conversation
…characters (new gTLDs)
But what about the new extensions to come, or already live? Like restaurant, photography, healthcare... Is there already a plan to cover them as soon as there are available? |
No interest in this?? I've rechecked the current list, export the data and the maximum length would be 15 strings for So I am setting the max to 15. |
Our regex in 3.x is now Shall we use the same? |
Well, maybe, but I do not use regex everyday, so I will track down the commits and comments to double check before doing a blind copy and paste 😎 |
Now a email address like contact@me.international works, as well as me.internationalandmore... So after scratching my head on this tonight, I still don't know exactly what was the purpose of the new regex, and there are many changes in the js folder in J3.3 compare to current J2.5.28-dev, plus the use of punycode.toASCII() here
So I would prefer that some devs who authored and committed (and hopefully understand ;-) the code involved, review this to help to back port it from J3.3, if possible. If this is too complicated or not possible, using the 15 strings limit from my previous PR may be enough for J2.5. What do you think? |
The new regex was specially crafted to take care of longer domains in the platform and ported to 3.x Therefore this PR is OK for me (with the limitations concerning INtl domains in 2.5, which is OK). |
hmm, weird |
We need to correct 3.x |
Please see and test (for 3.x) And please change your PR to use that regex |
Waouh, well done Jean-Marie @infograf768 for the head up!! I should have check with Regex from the code, and not only the only pasted in the comments. Lesson learnt!! PR updated now. Please test |
Thanks @infograf768 , I have updated ReGex and backported the helper as well (without the punnyCode of course). Please Test! |
Have you also backported |
Double checking now. Oops, wrong ReGex in validate.js, corrected now. But we do not have the $tld test on Joomla 2.5, so there is not similar part with this line IMHO Otherwise, backported, please Test! |
Hi, |
Hi Dan @docschmitti Thanks, |
Hi, |
Hi @docschmitti, Kind regards, |
HI, sorry. I am not familiar with programming. Can only do things a bit with copy/paste. Cannot handle all that Github stuff:-( I am afraid I cannot help... besides someone can give me a zip-file and I can install it... :-( |
One of our easiest things we can do 😄 |
ok, thank you. Looks like a complete package of joomla to me. What does it do? Or what is it? A new beta Joomla 2.5? Regards, |
@docschmitti this is a joomla 2.5 with the changed files by this PR. You can install it as you install joomla normaly and after this test all the things that need to be tested. Thanks 👍 |
ok, so I should NOT install it on a running website? Only for testing? |
@docschmitti NO please do not install any patches on a live/productive site. If the patch here have 2 test it get into the next update. The zip is for testing in a testing envoirment 😄 |
Test OK. Merging. |
You say don't put on a production site... however I am stuck using Joomla 2.5 as our component is not yes ready to move away from an old version of virtuemart. Now I have the problem on multiple sites that people can't register. Will there be a patch for joomla 2.5 that we can use? This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4135. |
Simple answer is no. There is no official support for Joomla 2.5 any more On 6 August 2015 at 09:58, davidmdawson notifications@github.com wrote:
Brian Teeman |
If this is related to the use of https://github.com/rvbgnu/joomla-cms/archive/2.5.x_fix_33948.zip, it is no need. This patch has been merged into 2.5.28 in December 2014 |
oh, I still have the problem. I'll look elsewhere then. Maybe an issue with the registration page in virtuemart This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4135. |
J2.5 email addresses with new TLD extensions are not valid upon installation
Steps to reproduce the issue
Install a new site with Joomla! 2.5.x (I believe it is in all versions).
Provide an admin email address with a new type of extension (e.g. .ninja .expert .company .cool .sexy etc.)
Test also any kind of email following these rules:
The local part of an email (text before the "@") may include any of the following characters
Uppercase and lowercase English letters (a–z, A–Z)
Digits 0 to 9
These special characters: ! # $ % & ’ * + - / = ? ^ _ ` { | } ~
Character . (dot, period, full stop) (ASCII: 46) provided that it is not the first or last character, and provided also that it does not appear consecutively (e.g. John..Doe@example.com is not allowed).
Expected result
Any email address from current and scheduled domain name should be accepted.
Actual result
Address is not accepted as valid.
System information (as much as possible)
Initial report on Joomla 2.5.20 - confirmed in 2.5.28-dev
Additional comments
Initial issue Submitted By: Ben Shomer - Open Date: 2014-07-14
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33948
fix email validation, allowing email with new domain name extension (more than 4 characters) like new gTLDs