[#19349] - Mail::sendMail() shouldn't use $autoReplyTo in Mail::setSender()
- Closed
- 11 May 2018
- Medium
- Build: staging
- # 19349
Steps to reproduce the issue
There's a hack in libraries/src/Mail/Mail.php:742:
If the replyTo field doesn't exist use third param in Mail::setSender().
The third param in Mail::setSender() (inherited from PHPMailer::setFrom()) had used for replyTo functional initially. But this functional has been cleaned from PHPMailer since 5.2.7 version (#51, PHPMailer::setFrom $auto should be FALSE by default).
Actual result
PHP mail() function doesn't send mails without the replyTo field in some conditions of mail servers. Mail servers can defer the mail with custom Sender field in some conditions.
System information (as much as possible)
Joomla since 3.6.4 to latest, the Mailer is mail, PHP since 5.4.45 to latest
| Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
| Status | New | ⇒ | Information Required |
I'm a not a joomla user/programmer. I'm a web/mail server admin only on which other people install Joomla.
I don't undertsand a joomla goals and architecture. I can not submit a pull request. :(
| Status | Information Required | ⇒ | Discussion |
@drTr0jan so we have to wait for a Developer.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19349.
brianteeman
- | Labels |
Added:
J3 Issue
|
||
Please confirm to omit the 3rd parameter from the call.
Before:
// Add sender to replyTo only if no replyTo received
$autoReplyTo = (empty($this->ReplyTo)) ? true : false;
if ($this->setSender(array($from, $fromName, $autoReplyTo)) === false)
{
return false;
}
After:
if ($this->setSender(array($from, $fromName)) === false)
{
return false;
}
| Status | Discussion | ⇒ | Information Required |
Issue closed as no Information provided since 1+ Month. Issue can always reopened.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19349.
| Status | Information Required | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-05-11 04:24:25 |
| Closed_By | ⇒ | franz-wohlkoenig |
| Closed_By | franz-wohlkoenig | ⇒ | joomla-cms-bot |
Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/19349
Please submit a PR.