| 23 | | The reality is that the problem comes, when people have forced their `sendmail` binary `from` for whatever reason. In this case, now two `From` addresses will be conflicting. Anyone who could have had troubles with the missing `Sender` address because of this bug, shouldn't have been using the default `wp_mail` behaviour if they pretended to add `-f` enabled in their sendmail. I understand that the Mail component has been a little abandoned for years and people were resorting to whatever worked the best. But instead, they should have switched to `isSendMail`. By disabling the `Sender` option here, the regular `mail` became too permissive and was enabling sending mails with regular `mail` wrongly (and this is not ideal according to the PHP docs). |
| | 23 | The reality is that the problem comes, when people have forced their `sendmail` binary `from` for whatever reason. In this case, now two `From` addresses will be conflicting. Anyone who could have had troubles with the missing `Sender` address because of this bug, shouldn't have been using the default `wp_mail` behaviour if they pretended to add `-f` enabled in their sendmail. I understand that the Mail component has been a little abandoned for years and people were resorting to whatever worked the best. But instead, they should have switched to `isSendMail`. By having the `Sender` option disabled here because of [38286], the regular `isMail` became too permissive and was enabling sending mails with PHP `mail` function wrongly (and this is not ideal according to the PHP docs). |