Bug #42106
closed
swiftmaileradapter should ignore empty headers
100%
Description
There are cases when for example Direct Mail (by error) sends an empty header, in this case "Reply-To:". That leads to an error like described in #27633 later with swiftmailer (Address in mailbox given [@localhost] does not comply with RFC 2822, 3.6.2).
In t3lib/mail/class.t3lib_mail_swiftmaileradapter.php inside setHeader() the headers are transfered over. Executing the line
$addressList = $this->parseAddresses($headerValue);
on the empty value leads to transfer the address to "@localhost" here.
Suggestion: trim $headerValue and check it is not empty before transfering it.
Updated by Stefan Neufeind about 12 years ago
I'm still looking into why Direct Mail generates an empty Reply-To-header anyhow. But, if by error, such a value slips in we could ignore it in the adapter.
Updated by Stefan Neufeind almost 12 years ago
- Assignee set to Stefan Neufeind
Already under review at: https://review.typo3.org/15779
Updated by Gerrit Code Review over 11 years ago
- Status changed from New to Under Review
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/15779
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/19685
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19686
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19686
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19687
Updated by Stefan Neufeind over 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ead60748fac94b1ca19f2ccfc8c91d09bb55a93c.
Updated by