Bug #69208: Mail address with UTF-8 smilie validates - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #69208

closed

Mail address with UTF-8 smilie validates

Added by aimcom no-lastname-given over 8 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Morton Jonuschat

Category:

Target version:

7.5

Start date:

2015-08-20

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.2

PHP Version:

5.5

Tags:

Complexity:

medium

Is Regression:

Sprint Focus:

Description

A mail address like "john.doe@☺gmail.com" will return TRUE if passed to GeneralUtility::validEmail(). But if I try to set it as sender or receiver it throws a Swift_RfcComplianceException "Address in mailbox given [john.doe@☺gmail.com] does not comply with RFC 2822, 3.6.2.".

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Morton Jonuschat over 8 years ago

Status changed from New to On Hold

GeneralUtility::validEmail() is actually correct as the smiley in the domain name is valid as long as it's punycode encoded. There's an open bug report/feature request regarding IDN Support for SwiftMailer: https://github.com/swiftmailer/swiftmailer/issues/63

We need to wait on SwiftMailer to fix this.

Actions

Copy link

Updated by Felix Heller over 8 years ago

At the TYPO3camp Munich 2015, we tried to fix this bug and thought that the best solution was to extend the default methods setTo, setFrom etc. in \TYPO3\CMS\Core\Mail\MailMessage\ and execute a IDNA encoding like in GeneralUtility::validEmail:

if (!preg_match('/^[a-z0-9.\\-]*$/i', $domain)) {
    $domain = self::idnaEncode($domain);
}

It seems like the IDNA encoded domain could pass the validation methods of SwiftMailer.

Actions

Copy link

Updated by Riccardo De Contardi over 8 years ago

Status changed from On Hold to New

Unlocked after discussion held on TYPO3camp Munich 2015;
\TYPO3\CMS\Core\Mail\MailMessage should be adapted to fix this problem

Actions

Copy link

Updated by Morton Jonuschat over 8 years ago

Status changed from New to In Progress
Assignee set to Morton Jonuschat
Target version set to 7.5
Sprint Focus set to On Location Sprint

Actions

Copy link

Updated by Gerrit Code Review over 8 years ago

Status changed from In Progress 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 http://review.typo3.org/43294

Actions

Copy link

Updated by Gerrit Code Review over 8 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/43294

Actions

Copy link

Updated by Gerrit Code Review over 8 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/43294

Actions

Copy link

Updated by Gerrit Code Review over 8 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/43294

Actions

Copy link