Project

General

Profile

Actions

Feature #56771

closed

Additional check for valid domain in validEmail()

Added by Benni Mack about 10 years ago. Updated over 9 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
Category:
Miscellaneous
Target version:
Start date:
2014-03-11
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

If using a correct email address (= correct syntax),
but the domain name has no valid DNS entry,
then a given email is validated as valid,
but the destination is still wrong. In some cases
the email should be completely validated.
By adding an additional PHP call to check if
the DNS entry is valid, the validEmail() can
be put in a stricter mode if called explicitly.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #64064: Revert #56771 which introduced brittle feature + tests to validEmail() which were beyond the scope of the functionClosedSteffen Müller2014-12-30

Actions
Actions #1

Updated by Gerrit Code Review about 10 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28264

Actions #2

Updated by Gerrit Code Review over 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/28264

Actions #3

Updated by Gerrit Code Review over 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/28264

Actions #4

Updated by Gerrit Code Review over 9 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/28264

Actions #5

Updated by Benni Mack over 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #6

Updated by Steffen Müller over 9 years ago

Will be reverted with #64064
Please use this ticket #64064 for comments.

Actions #7

Updated by Steffen Müller over 9 years ago

  • Status changed from Resolved to Accepted
  • Target version changed from 6.2.0 to 7.1 (Cleanup)
  • TYPO3 Version changed from 6.2 to 7
Actions #8

Updated by Steffen Müller over 9 years ago

  • Tracker changed from Bug to Feature
Actions #9

Updated by Steffen Müller over 9 years ago

How to proceed with this ticket? I doubt we need this feature in the core, since it can be achieved with a basic php function.

Actions #10

Updated by Benni Mack over 9 years ago

  • Status changed from Accepted to Rejected

We leave it for now (rejected), and the customer should implement it on his own. :(

Actions

Also available in: Atom PDF