Feature #87691

Allow email adresses with "+" in GeneralUtility::validEmail()

Added by Daniel Koether 3 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2019-02-11
Due date:
% Done:

0%

PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Hello everyone,

as a person who uses email addresses like for test purposes all day I would like to ask if there is a reason why "+" is not allowed in \TYPO3\CMS\Core\Utility\GeneralUtility::validEmail()?

See for more informations:
https://www.cs.rutgers.edu/~watrous/plus-signs-in-email-addresses.html

Thanks in advance!

Best regards
Daniel


Related issues

Related to TYPO3 Core - Bug #73616: Different standards used in GeneralUtility::validEmail (RFC 3696) and swiftmailers Swift_Mime_Grammar-class (RFC 2822) New 2016-02-23

History

#1 Updated by Andreas Kiessling 3 months ago

Hi Daniel,

Which TYPO3 version are you using? AFAIR, the validEmail method has changed some time ago, + should not be a problem in the local part
There is also a unit test which covers that: https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/core/Tests/Unit/Utility/GeneralUtilityTest.php#L915

#2 Updated by Georg Ringer 3 months ago

  • Related to Bug #73616: Different standards used in GeneralUtility::validEmail (RFC 3696) and swiftmailers Swift_Mime_Grammar-class (RFC 2822) added

#3 Updated by Georg Ringer 3 months ago

  • Status changed from New to Needs Feedback

#4 Updated by Daniel Koether 3 months ago

Andreas Kiessling wrote:

Hi Daniel,

Which TYPO3 version are you using? AFAIR, the validEmail method has changed some time ago, + should not be a problem in the local part
There is also a unit test which covers that: https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/core/Tests/Unit/Utility/GeneralUtilityTest.php#L915

Thanks for answering!

Sorry for the inconvenience, it seems like there was an issue with multiple validators in my form of EXT:form. The special character + inside email addresses is now valid and everything works as wanted. This issue can be closed.

#5 Updated by Riccardo De Contardi 3 months ago

  • Status changed from Needs Feedback to Closed

@Andreas Kiessling thank you for your quick answer.

Closed as requested by the reporter.

Also available in: Atom PDF