User tests: Successful: Unsuccessful:
Pull Request resolves #46419 .
Allow to configure the email for Sender property of PHPMailer. This is used for Return Path.
When it is not present then the email of the server will be used. This leads to broken validation for SPF / DKIM settings.
This is replaces for:
Note: It may not work the same for all servers.
Example via "Google SMTP" Google will force account email as Return Path
Apply path.
Also please check testing description from #46431
Send test email with and without Sender.
And check e-mail headers of the received emails.
Return path always contain server email.
Return path contain email you set in Sender field.
Please select:
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_config Language & Strings JavaScript Repository NPM Change Libraries |
I just tested this during Friday PR Testing Group, and sadly I do not see it or understand it as working...
Probably due to my environment... or our testing.
What I do see is BEFORE and AFTER THE PR, there are 2 Return-Paths, one as Return-Path: and one as
Return-path:
The testing instructions also do not talk about the Sender Email (optional) field...
Perhaps some screenshots would be very helpful - thank you so much.
| Labels |
Added:
Feature
Language Change
NPM Resource Changed
PR-6.2-dev
|
||
Probably due to my environment... or our testing.
Yes it is, it hard to make a test that will work for all.
This is very environment depend issue also very depend from used email provider.
The testing instructions also do not talk about the Sender Email (optional) field...
The field is a new feature in this PR.
Originally the fix was to use the same "Email From" also for "Sender Email", but turns out it is not very flexible.
I tested this locally using SMTP(Mailpit)
Before PR: on 6.2.0-alpha1-dev
After PR : I am using the prebuilt : 6.2.0-alpha1-dev+pr.47435
my setup:

Results on "Send test mail"

The Return-Path is same in the From address in both scenarios, ignoring the custom Sender email configuration. Is it possible the Test Email controller isn't fetching the new mail_sender param?
If my test needs some modification or I did anything wrong - let me know.
I think your SMTP does not allow customization of the return path in current configuration
This shouldn't concern smtp but phpmailer, right? With smtp the sender is always the smtp account. Will look and test it next week.
This shouldn't concern smtp but phpmailer, right?
It depend from what the email server allows you to do.
I'm very confused about this PR and more confused after reading this https://www.postmastery.com/about-the-return-path-header/
how would you recommend we best verify this PR locally?
You can check the property in PHP, like:
$mailer = Factory::getContainer()->get(MailerFactoryInterface::class)->createMailer();
dump($mailer->Sender);This should show email you set in the configuration.
I'm very confused about this PR and more confused after reading this
The article from 2010, a lot has changed since then, a LOT.
Look.
You have website brian.website
Hosted on potato.hosting
When you send email from Joomla your email will contain:
From: email@brian.website
Return Path account123.email@potato.hosting
When email failed to delivery the email will bounce back to account123.email@potato.hosting
Additionally the SMTP server can signing your email with SPF/DKIM.
To do this it check the email domain (from Return Path) and DNS records for that domain, when there a match and DNS has a correct record then your email will be signed.
But because Return Path is email@potato.hosting and you cannot configure it in Joomla the signing will not work.
Also a lot depend from the Hosting itself, I mean from SMTP provider you use.
This bug report helped me understand this a little better PHPMailer/PHPMailer#2298
@TLWebdesign please check