X-mailer and X-priority headers can impact the quality of email for various delivery systems and increase the chances that a message will be marked as spam. The SMPT module should only set these headers when it is configured to operate in debug mode.

Original report by @VanHomero

The module add in X-mailer header the version of PHPMailer.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

VanHomero created an issue. See original summary.

VanHomero’s picture

Add the X-Mailer header only if Debugger is active.

VanHomero’s picture

Fix patch comment 2.

VanHomero’s picture

To improve the quality of the mail, disable X-Mailer with PHPMailer source and X-Priority.

VanHomero’s picture

asrob’s picture

Status: Active » Needs review
dww’s picture

Title: X-mailer header add PHPMailer version » X-mailer header for PHPMailer version should only be set in debug mode
Issue summary: View changes
Related issues: +#2711559: Set phpMailer as a external library using composer (and update it to 6.0)

Improved the title and issue summary to make it clear what this bug report is about. At first glance, I had no idea what this was for.

However, the code in question is actually from an external library. I think the better solution is #2711559: Set phpMailer as a external library using composer (and update it to 6.0). Probably the latest version of the library fixes this behavior. If not, this really belongs as an issue upstream, not here.

Chris Matthews’s picture

Status: Needs review » Closed (won't fix)
Issue tags: -smtp, -x-mailer

Agree with @dww's comments in #7 so closing this 2 year old issue in favor of #2711559: Set phpMailer as a external library using composer (and update it to 6.0).