Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description (*)
Laminas SMTP Transport Options has the option to set the name of the local client hostname, this is the name the client uses to identify itself to the mail server (
HELO
command).The default setting is
localhost
, but some spam filters don't like that (SpamAssassin ruleHELO_LOCALHOST
)This patch uses the hostname of the website as name in the SmtpOptions.
Manual testing scenarios (*)
Received: from localhost (real.host.name [1.2.3.4]) by mx.mail.server;
Received: from store.host.name (real.host.name [1.2.3.4]) by mx.mail.server;
Questions or comments
Should this be configurable? Are there situations where this solution is worse than
localhost
?